-
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 API gateway adapter common module and Spring Cloud Gateway adapter #695
Conversation
Codecov Report
@@ Coverage Diff @@
## master #695 +/- ##
============================================
- Coverage 41.38% 41.28% -0.11%
- Complexity 1236 1319 +83
============================================
Files 266 288 +22
Lines 7969 8478 +509
Branches 1091 1147 +56
============================================
+ Hits 3298 3500 +202
- Misses 4258 4545 +287
- Partials 413 433 +20
Continue to review full report at Codecov.
|
20d7aac
to
d578e0a
Compare
…rule and API definition management Signed-off-by: Eric Zhao <sczyh16@gmail.com>
Signed-off-by: Eric Zhao <sczyh16@gmail.com>
Signed-off-by: Eric Zhao <sczyh16@gmail.com>
aef56d3
to
a0deec5
Compare
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
Describe what this PR does / why we need it
Add an API gateway adapter common module, as well as Sentinel Spring Cloud Gateway adapter module and implementation.
Does this pull request fix one issue?
Resolves #86
Describe how you did it
sentinel-api-gateway-adapter-common
module for universal gateway rule and API definition managementsentinel-api-gateway-adapter-common
andsentinel-reactor-adapter
)Describe how to verify it
Run the test cases and demo.
Special notes for reviews
NONE