Skip to content

optimize: upgrade tomcat to 9.0.106#7488

Merged
funky-eyes merged 2 commits intoapache:2.xfrom
Dltmd202:7472
Jul 2, 2025
Merged

optimize: upgrade tomcat to 9.0.106#7488
funky-eyes merged 2 commits intoapache:2.xfrom
Dltmd202:7472

Conversation

@Dltmd202
Copy link
Contributor

@Dltmd202 Dltmd202 commented Jul 1, 2025

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

fixes #7472

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@YongGoose YongGoose added the first-time contributor first-time contributor label Jul 1, 2025
@YongGoose
Copy link
Member

@YongGoose
Copy link
Member

Also, I think we should upgrade version to 9.0.106 :)

image

@codecov
Copy link

codecov bot commented Jul 1, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.37%. Comparing base (bf6601f) to head (5171bd7).
Report is 1 commits behind head on 2.x.

Additional details and impacted files
@@             Coverage Diff              @@
##                2.x    #7488      +/-   ##
============================================
+ Coverage     60.35%   60.37%   +0.02%     
  Complexity      658      658              
============================================
  Files          1284     1284              
  Lines         48465    48465              
  Branches       5694     5694              
============================================
+ Hits          29249    29261      +12     
+ Misses        16596    16586      -10     
+ Partials       2620     2618       -2     

see 5 files with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Dltmd202
Copy link
Contributor Author

Dltmd202 commented Jul 1, 2025

Thanks for the guidance! 🙌
I'll make the updates to those

@YongGoose
Copy link
Member

YongGoose commented Jul 1, 2025

Thanks for the guidance! 🙌 I'll make the updates to those

Once the PR is ready for review, feel free to ping me

@Dltmd202 Dltmd202 marked this pull request as ready for review July 1, 2025 16:15
@Dltmd202
Copy link
Contributor Author

Dltmd202 commented Jul 2, 2025

Hi @YongGoose ,
I’ve made the updates. When you get a chance, could you please take another look? 🙏

@YongGoose YongGoose changed the title optimize: upgrade tomcat to 9.0.71 optimize: upgrade tomcat to 9.0.106 Jul 2, 2025
Copy link
Member

@YongGoose YongGoose left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@funky-eyes funky-eyes added this to the 2.5.0 milestone Jul 2, 2025
@funky-eyes funky-eyes added the dependencies Pull requests that update a dependency file label Jul 2, 2025
Copy link
Contributor

@funky-eyes funky-eyes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@funky-eyes funky-eyes merged commit db73418 into apache:2.x Jul 2, 2025
10 checks passed
slievrly pushed a commit to slievrly/fescar that referenced this pull request Oct 21, 2025
YvCeung pushed a commit to YvCeung/incubator-seata that referenced this pull request Dec 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file first-time contributor first-time contributor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

upgrade apache tomcat dependency

3 participants