Skip to content

chore(deps): constrain test-scope Tomcat to 10.1.61 past GHSA-f525-44xv-f2qj - #58

Merged
rennf93 merged 1 commit into
masterfrom
fix/tomcat-ghsa-f525
Nov 3, 2026
Merged

rennf93 merged 1 commit into
masterfrom
fix/tomcat-ghsa-f525

Conversation

@rennf93

@rennf93 rennf93 commented Nov 3, 2026

Copy link
Copy Markdown
Member

The osv-scanner job is red on master: the Boot 3.5.16 BOM manages tomcat-embed-core at 10.1.55, and GHSA-f525-44xv-f2qj (CVSS 9.8, fixed in 10.1.59) has no suppression. That is the Dependency audit failure on #57.

Tomcat is test scope only (spring-boot-starter-web under testImplementation) and not in the adapter POM, so a testImplementation constraint moves tomcat-embed-core, -el and -websocket to 10.1.61. OSV lists no advisory for any of the three at 10.1.61, and 10.1.55 also carried GHSA-9xv2-5v5q-p794, GHSA-gcx9-497g-6cp6 and GHSA-h3x4-894j-xpx5, which the config suppressed. Those three suppressions come out of osv-scanner.toml in this PR, as its header asks once the dependency moves. gradle.lockfile changes only the three Tomcat lines.

Locally, osv-scanner v2.6.0 (the CI binary's release, checksum checked) with the CI command reports "No issues found" here and fails on master with GHSA-f525-44xv-f2qj. Gradle resolves testCompileClasspath and testRuntimeClasspath against the lock with no failures, while the old 10.1.55 lines with the constraint in place fail resolution. spotlessCheck, detekt and build -x test pass. I didn't run the tests here because they hard-code localhost:6379, so this PR's own build job is the test run.

…xv-f2qj

The Boot 3.5.16 BOM manages tomcat-embed-core, -el and -websocket at 10.1.55, which carries GHSA-f525-44xv-f2qj (CVSS 9.8, fixed in 10.1.59) and three more advisories the config suppressed. A testImplementation constraint moves the three modules to 10.1.61, the lock changes only those three lines, and the three Tomcat suppressions come out of osv-scanner.toml in the same change, as its header asks once the dependency moves.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01E28c6iftj4hRmx55Uaaoxc
Entire-Checkpoint: 01M6FARB3G4EA27Z272N75MDC9
@rennf93 rennf93 added the no-issue Chore or dependency PR that does not need an issue label Nov 3, 2026
@rennf93
rennf93 merged commit 37d1fa0 into master Nov 3, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-issue Chore or dependency PR that does not need an issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant