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

[#1099] adjusting new logic and configuration for security #1100

Merged
merged 4 commits into from
Nov 22, 2023

Conversation

chengyouling
Copy link
Collaborator

@chengyouling chengyouling commented Nov 16, 2023

1、策略uri配置/*时,客户端对应方法类型的请求都通过;
2、开启安全策略校验,但是没有设置策略或者安全策略白名单为空时,打印告警日志(发送告警信息),宽容模式请求通过,强制模式,请求拦截;
3、header中获取token的key调整为支持配置,默认为X-SM-Token。

@chengyouling chengyouling self-assigned this Nov 16, 2023
@codecov-commenter
Copy link

codecov-commenter commented Nov 21, 2023

Codecov Report

Attention: 15 lines in your changes are missing coverage. Please review.

Comparison is base (2f248ce) 0.00% compared to head (08f5512) 0.00%.
Report is 4 commits behind head on master.

❗ Current head 08f5512 differs from pull request most recent head 8263b42. Consider uploading reports for the commit 8263b42 to get more accurate results

Files Patch % Lines
...iscovery/graceful/ServicecombGracefulEndpoint.java 0.00% 12 Missing ⚠️
...graceful/ServicecombGracefulAutoConfiguration.java 0.00% 2 Missing ⚠️
...ain/java/com/huaweicloud/config/ConfigService.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##           master   #1100   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files         117     119    +2     
  Lines        2276    2290   +14     
  Branches      194     196    +2     
======================================
- Misses       2276    2290   +14     

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

@chengyouling chengyouling merged commit ebb6b4e into master Nov 22, 2023
1 check passed
@chengyouling chengyouling deleted the master_secnew branch November 27, 2023 03:52
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.

2 participants