-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
security: upgrade spring mvc and tomcat.embed #6339
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 2.x #6339 +/- ##
============================================
- Coverage 51.94% 51.93% -0.02%
+ Complexity 5186 5184 -2
============================================
Files 921 921
Lines 32166 32166
Branches 3874 3874
============================================
- Hits 16710 16705 -5
- Misses 13821 13824 +3
- Partials 1635 1637 +2 |
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
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
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
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 did
upgrade spring mvc and tomcat.embed
Ⅱ. Does this pull request fix one issue?
tomcat:
MPS-uy56-j8e4
MPS-b5of-dwyh
MPS-hz9y-jtfe
MPS-67md-8wr2
spring mvc:
MPS-2022-62832
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews