-
Notifications
You must be signed in to change notification settings - Fork 8k
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
Feature: support registering writable data-source for GatewayFlowRule and customized ApiDefinition #1057
Conversation
Could you please sign the CLA here: https://cla-assistant.io/alibaba/Sentinel?pullRequest=1057 |
Codecov Report
@@ Coverage Diff @@
## master #1057 +/- ##
===========================================
- Coverage 43.1% 43.01% -0.1%
+ Complexity 1493 1492 -1
===========================================
Files 317 317
Lines 9295 9321 +26
Branches 1269 1271 +2
===========================================
+ Hits 4007 4009 +2
- Misses 4794 4818 +24
Partials 494 494
Continue to review full report at Codecov.
|
ok |
...com/alibaba/csp/sentinel/adapter/gateway/common/command/UpdateGatewayRuleCommandHandler.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
...com/alibaba/csp/sentinel/adapter/gateway/common/command/UpdateGatewayRuleCommandHandler.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks for contributing! |
…omized ApiDefinition (alibaba#1057)
…libaba#1177) * Filter out docker0 when finding a local address * sweep out bridge interface Co-authored-by: yuanyuan.xia <yuanyuan.xia@envisioncn.com>
Describe what this PR does / why we need it
Does this pull request fix one issue?
Resolves #992
Describe how you did it
Describe how to verify it
Special notes for reviews