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
fix bug about webflux broken trace #5335
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5335 +/- ##
============================================
+ Coverage 53.13% 53.41% +0.27%
- Complexity 3204 3211 +7
============================================
Files 828 828
Lines 20830 20768 -62
Branches 2019 2006 -13
============================================
+ Hits 11069 11093 +24
+ Misses 8856 8768 -88
- Partials 905 907 +2
Continue to review full report at Codecov.
|
@@ -79,6 +78,7 @@ public Object afterMethod(EnhancedInstance objInst, Method method, Object[] allA | |||
return ((Mono) ret).doFinally(s -> { | |||
AbstractSpan span = (AbstractSpan) exchange.getAttributes().get("SKYWALING_SPAN"); | |||
if (span != null) { | |||
ContextManager.stopSpan(span); |
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.
I double this is a fix. This should be a part of another thread.
@ascrutae I will wait for your confirmation. |
@ascrutae I close this for now. Wait for your fix. |
Is this webflux use @requestbody with problem solved? i have same question, Can you give me some ideas? |
Please answer these questions before submitting a pull request
Why submit this pull request?
Bug fix
New feature provided
Improve performance
Related issues
Bug fix
the trace of webflux application which use @RequestBody is broken #5184
the trace of webflux application which use @requestbody is broken
skywalking has some problem with webflux #4957
skywalking has some problem with webflux
I put the stopSpan operation in DispatcherHandlerHandleMethodInterceptor from beforeMethod to afterMethod
New feature or improvement