-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
Add rule configuration support for parameter flow control in Sentinel Dashboard #176
Conversation
- Upgrade Spring Boot to 2.0.x - Remove unused dependencies Signed-off-by: Eric Zhao <sczyh16@gmail.com>
Signed-off-by: Eric Zhao <sczyh16@gmail.com>
Signed-off-by: Eric Zhao <sczyh16@gmail.com>
Signed-off-by: Eric Zhao <sczyh16@gmail.com>
Signed-off-by: Eric Zhao <sczyh16@gmail.com>
Signed-off-by: Eric Zhao <sczyh16@gmail.com>
Codecov Report
@@ Coverage Diff @@
## master #176 +/- ##
============================================
+ Coverage 50.47% 51.55% +1.07%
- Complexity 817 836 +19
============================================
Files 139 139
Lines 4741 4799 +58
Branches 675 678 +3
============================================
+ Hits 2393 2474 +81
+ Misses 2067 2033 -34
- Partials 281 292 +11
Continue to review full report at Codecov.
|
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
… Dashboard (alibaba#176) - Update dashboard API client to support authority and parameter flow commands - Add REST API controller for parameter flow rules in dashboard - Add pages for configuring parameter flow rules - Update dependencies: upgrade Spring Boot to 2.x and remove unused dependencies - Other refinements Signed-off-by: Eric Zhao <sczyh16@gmail.com>
Describe what this PR does / why we need it
Add rule configuration support for parameter flow control in Sentinel Dashboard.
Does this pull request fix one issue?
Resolves #175
Describe how you did it
Describe how to verify it
Run the dashboard to verify the functions.
Special notes for reviews
None