Contributor

Purpose of the PR

• close #xxx

Main Changes

Verifying these changes

• Trivial rework / code cleanup without any test coverage. (No Need)
• Already covered by existing tests, such as AuthTest.
• Need tests and can be verified as follows:
  • xxx

Does this PR potentially affect the following parts?

• Dependencies (add/update license info & regenerate_dependencies)
• Modify configurations
• The public API
• Other affects (typed here)
• Nope

Documentation Status

• Doc - TODO
• Doc - Done
• Doc - No Need

fix(server): random generate default jwt secret key

1460469

Merge branch 'apache:master' into master

9840c7c

imbajin reviewed Jul 13, 2024

View reviewed changes

View reviewed changes

Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codecov Report

Attention: Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.

Project coverage is 4.56%. Comparing base (37e71c4) to head (85c2b6d).
Report is 2 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (37e71c4) and HEAD (85c2b6d). Click for more details.

HEAD has 5 uploads less than BASE

Flag	BASE (37e71c4)	HEAD (85c2b6d)
	6	1

@@             Coverage Diff              @@
##             master   #2568       +/-   ##
============================================
- Coverage     56.42%   4.56%   -51.87%     
+ Complexity      586     479      -107     
============================================
  Files           612     621        +9     
  Lines         49672   50875     +1203     
  Branches       6681    6570      -111     
============================================
- Hits          28029    2321    -25708     
- Misses        18833   48367    +29534     
+ Partials       2810     187     -2623     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

VGalaxies approved these changes Jul 14, 2024

View reviewed changes