Skip to content
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

try to address the issue with CVE-2025-24813 spotted via https://nvd.… #323

Closed
wants to merge 1 commit into from

Conversation

aeggerd
Copy link

@aeggerd aeggerd commented Mar 26, 2025

Hi Philip,
our IT security department found the following vulnerability in the latest version.
https://nvd.nist.gov/vuln/detail/CVE-2025-24813

i tried to address it using the following guide:
https://dev.to/carrie_luo1/apache-tomcat-partial-put-remote-code-execution-vulnerability-cve-2025-24813-1pae

Could you might review the changes that i made to the web.xml files.
Get back to me if you have any more questions regarding this.
Best Michael

@phax
Copy link
Owner

phax commented Mar 27, 2025

Hello @aeggerd thanks for providing the PR.
However, your solution limits the SMP to the Tomcat application server by introducing a specific dependency.
I already tackled that CVE in #319 by updating and re-creating the Docker images which lead to an updated image with an up-to-date Tomcat version.

Additionally, the path / is already mapped to class SMPRootServlet so the Tomcat default servlet will not kick in.

The update of Tomcat itself is something that each SMP users needs to do on its own.

If you agree, I would not merge your PR based on the arguments mentioned above.

hth, Philip

@phax phax closed this Mar 29, 2025
@aeggerd
Copy link
Author

aeggerd commented Mar 30, 2025

Awesome that will definitely work as well :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants