Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[type:fix] fix JwtUtils #4420

Merged
merged 1 commit into from
Mar 2, 2023
Merged

[type:fix] fix JwtUtils #4420

merged 1 commit into from
Mar 2, 2023

Conversation

moremind
Copy link
Member

@moremind moremind commented Mar 2, 2023

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@moremind moremind added this to the 2.6.0 milestone Mar 2, 2023
@codecov-commenter
Copy link

codecov-commenter commented Mar 2, 2023

Codecov Report

Merging #4420 (8dc0391) into master (18b3ac1) will decrease coverage by 0.07%.
The diff coverage is 37.75%.

@@             Coverage Diff              @@
##             master    #4420      +/-   ##
============================================
- Coverage     68.09%   68.02%   -0.07%     
- Complexity     7483     7491       +8     
============================================
  Files          1021     1022       +1     
  Lines         28913    28992      +79     
  Branches       2583     2592       +9     
============================================
+ Hits          19689    19723      +34     
- Misses         7683     7721      +38     
- Partials       1541     1548       +7     
Impacted Files Coverage Δ
...g/apache/shenyu/common/cache/WindowTinyLFUMap.java 0.00% <0.00%> (ø)
.../org/apache/shenyu/common/config/ShenyuConfig.java 68.89% <33.33%> (-1.97%) ⬇️
...pache/shenyu/plugin/base/AbstractShenyuPlugin.java 44.59% <38.63%> (+0.33%) ⬆️
...n/java/org/apache/shenyu/admin/utils/JwtUtils.java 77.77% <100.00%> (ø)
.../plugin/base/cache/CommonPluginDataSubscriber.java 73.39% <100.00%> (+0.75%) ⬆️
...pache/shenyu/plugin/base/cache/MatchDataCache.java 100.00% <100.00%> (ø)
...e/shenyu/web/controller/LocalPluginController.java 94.65% <100.00%> (+0.02%) ⬆️
...henyu/admin/service/impl/UpstreamCheckService.java 66.66% <0.00%> (-1.15%) ⬇️
...henyu/common/cache/MemorySafeWindowTinyLFUMap.java 73.33% <0.00%> (ø)
...henyu/plugin/grpc/resolver/ShenyuNameResolver.java 61.70% <0.00%> (+5.31%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@KevinClair KevinClair merged commit 3a505a5 into apache:master Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants