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
support collecting feign params #5235
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5235 +/- ##
============================================
- Coverage 53.11% 52.37% -0.75%
+ Complexity 3165 3145 -20
============================================
Files 821 820 -1
Lines 20524 20521 -3
Branches 1974 1974
============================================
- Hits 10902 10748 -154
- Misses 8739 8900 +161
+ Partials 883 873 -10 Continue to review full report at Codecov.
|
...c/main/java/org/apache/skywalking/apm/plugin/feign/http/v9/DefaultHttpClientInterceptor.java
Outdated
Show resolved
Hide resolved
e3ef7a5
to
c3f1283
Compare
...-scenario/src/main/java/org/apache/skywalking/apm/testcase/feign/controller/RestRequest.java
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. Waiting for CI to confirm.
Welcome to be 251st contributor. |
Please answer these questions before submitting pull request
Why submit this pull request?
Bug fix
New feature provided
Improve performance
Related issues
feign-plugin support collecting feign params #5233