-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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 issue #5795 #5797
Fix issue #5795 #5797
Conversation
Fix the issue: nutz plugin throw NPE at afterMethod, cause traceid keep unchange for the thread
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.
Thanks for contributing, please fix codestyle(you can run ./mvnw checkstyle:check
locally to verify), and update the CHANGES.md
Thanks for the reply. Fixed codestyle and update the CHANGES.md. |
Codecov Report
@@ Coverage Diff @@
## master #5797 +/- ##
============================================
- Coverage 52.74% 0.00% -52.75%
============================================
Files 892 166 -726
Lines 22406 3991 -18415
Branches 2141 458 -1683
============================================
- Hits 11819 0 -11819
+ Misses 9637 3991 -5646
+ Partials 950 0 -950 Continue to review full report at Codecov.
|
Fix the issue: nutz plugin throw NPE at afterMethod, cause traceid keep unchange for the thread
Fix
Add a unit test to verify that the fix works.
Explain briefly about why the bug exists and how to fix it.
Closes nutz plugin throw NPE at afterMethod, cause traceid keep unchange for the thread #5795.
Update the
CHANGES
log.