Skip to content

bug: pin embedded Tomcat to 8.5.99 (CVE-2024-24549) - #477

Open
devin-ai-integration[bot] wants to merge 1 commit into
masterfrom
devin/sfind-15b80351-tomcat-cve-2024-24549
Open

devin-ai-integration[bot] wants to merge 1 commit into
masterfrom
devin/sfind-15b80351-tomcat-cve-2024-24549

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Summary

Fixes Snyk finding SNYK-JAVA-ORGAPACHETOMCATEMBED-6435948 / CVE-2024-24549 (HTTP/2 header-limit DoS, high, CVSS 7.5) in org.apache.tomcat.embed:tomcat-embed-core@8.5.15. UserFront gets it transitively from spring-boot-starter-web (Boot 1.5.4.RELEASE parent).

 <properties>
   <java.version>1.8</java.version>
+  <tomcat.version>8.5.99</tomcat.version>
 </properties>

Verification

The automation network policy blocks Maven Central and api.snyk.io. Boot 1.5.4 and Tomcat 8.5.99 aren't in the VM's Maven cache, so this exact pom could not be built, the existing tests could not run, and Snyk could not be re-run. I asked for network access; it was still pending when this PR was opened.

  • Snyk before/after issue count: not measured. Going by Snyk's own advisory data (vulnerable [8.5.0,8.5.99), fixed in 8.5.99), this PR clears this finding (1 -> 0 for SNYK-JAVA-ORGAPACHETOMCATEMBED-6435948). Someone with network access still needs to run snyk test --file=UserFront/pom.xml to confirm the total count.
  • Stand-in check: a minimal Boot 1.5.8 (Spring 4.3.12) spring-boot-starter-web app on JDK 11. With its default Tomcat 8.5.23 jars it serves requests (Starting Servlet Engine: Apache Tomcat/8.5.23). With the newer 8.5.31 jars swapped onto the classpath it also serves requests (Apache Tomcat/8.5.31). That shows Boot 1.5.x runs on a newer Tomcat 8.5 patch release, but 8.5.99 itself was not booted.

Devin-Org: engineering

Link to Devin session: https://devin-gtm.devinenterprise.com/sessions/d3eb0ed581874ce89567c74976c5a035
Open in Devin Desktop: https://devin-gtm.devinenterprise.com/desktop/session/d3eb0ed581874ce89567c74976c5a035?variant=devin


Devin Review

Status Commit
⚪ Not started —

Run Devin Review

Devin Review (Staging)


Devin Review

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.

0 participants