shenyu icon indicating copy to clipboard operation
shenyu copied to clipboard

[ISSUE #3713]Add new plugin: key-auth

Open Albertsirius opened this issue 3 years ago • 4 comments
trafficstars

#3713 Add new plugin: key-auth

Make sure that:

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

Albertsirius avatar Jul 25 '22 12:07 Albertsirius

pls check ci

yu199195 avatar Jul 26 '22 03:07 yu199195

pls check ci 屏幕截图 2022-07-26 151141 屏幕截图 2022-07-26 151238

I don't know why it failed. I didn't modify this source.

Albertsirius avatar Jul 26 '22 07:07 Albertsirius

Hi, @yu199195, it hasn't been reviewed, any problems?

Albertsirius avatar Aug 04 '22 01:08 Albertsirius

Codecov Report

Merging #3756 (b1aee90) into master (3e2b1a7) will increase coverage by 0.00%. The diff coverage is 77.55%.

@@            Coverage Diff            @@
##             master    #3756   +/-   ##
=========================================
  Coverage     70.17%   70.17%           
- Complexity     7012     7035   +23     
=========================================
  Files           954      958    +4     
  Lines         26315    26363   +48     
  Branches       2354     2362    +8     
=========================================
+ Hits          18467    18501   +34     
- Misses         6422     6433   +11     
- Partials       1426     1429    +3     
Impacted Files Coverage Δ
...ommon/dto/convert/rule/impl/KeyAuthRuleHandle.java 43.75% <43.75%> (ø)
...g/apache/shenyu/plugin/key/auth/KeyAuthPlugin.java 90.00% <90.00%> (ø)
...ava/org/apache/shenyu/common/enums/PluginEnum.java 100.00% <100.00%> (ø)
...gin/key/auth/handler/KeyAuthPluginDataHandler.java 100.00% <100.00%> (ø)
...er/plugin/key/auth/KeyAuthPluginConfiguration.java 100.00% <100.00%> (ø)
...va/org/apache/shenyu/plugin/cache/CachePlugin.java 81.57% <0.00%> (-7.90%) :arrow_down:
...che/shenyu/sync/data/http/HttpSyncDataService.java 85.71% <0.00%> (-4.09%) :arrow_down:
...ruptor/RegisterClientServerDisruptorPublisher.java 64.70% <0.00%> (+11.76%) :arrow_up:
...controller/ShenyuClientHttpRegistryController.java 100.00% <0.00%> (+22.22%) :arrow_up:

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

codecov-commenter avatar Sep 05 '22 14:09 codecov-commenter

@yu199195 Have modified and fixed the SQL scripts. Pls reviewed.

Albertsirius avatar Sep 17 '22 01:09 Albertsirius

@yu199195 Fixed the conflicting files

Albertsirius avatar Oct 22 '22 00:10 Albertsirius