fix(security): bump jetty-http 12.1.6 → 12.1.7 + pac4j-core 5.7.0 → 5.7.10 (1.13)#27980
fix(security): bump jetty-http 12.1.6 → 12.1.7 + pac4j-core 5.7.0 → 5.7.10 (1.13)#27980sonika-shah merged 3 commits into1.13from
Conversation
…ce (#27372) Bumps org.eclipse.jetty:jetty-http from 12.1.6 to 12.1.7. --- updated-dependencies: - dependency-name: org.eclipse.jetty:jetty-http dependency-version: 12.1.7 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: sonika-shah <58761340+sonika-shah@users.noreply.github.com>
…27373) Bumps org.eclipse.jetty:jetty-http from 12.1.6 to 12.1.7. --- updated-dependencies: - dependency-name: org.eclipse.jetty:jetty-http dependency-version: 12.1.7 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: sonika-shah <58761340+sonika-shah@users.noreply.github.com>
|
The Python checkstyle failed. Please run You can install the pre-commit hooks with |
Bumps [org.pac4j:pac4j-core](https://github.com/pac4j/pac4j) from 5.7.0 to 5.7.10. - [Commits](pac4j/pac4j@pac4j-parent-5.7.0...pac4j-parent-5.7.10) --- updated-dependencies: - dependency-name: org.pac4j:pac4j-core dependency-version: 5.7.10 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Chirag Madlani <12962843+chirag-madlani@users.noreply.github.com> Co-authored-by: Harsh Vador <58542468+harsh-vador@users.noreply.github.com>
Code Review ✅ ApprovedBumps jetty-http to 12.1.7 in service and mcp child modules to resolve the critical HTTP Request Smuggling vulnerability. No issues found. OptionsDisplay: compact → Showing less information. Comment with these commands to change:
Was this helpful? React with 👍 / 👎 | Gitar |
|
The Python checkstyle failed. Please run You can install the pre-commit hooks with |
|
🟡 Playwright Results — all passed (15 flaky)✅ 3740 passed · ❌ 0 failed · 🟡 15 flaky · ⏭️ 79 skipped
🟡 15 flaky test(s) (passed on retry)
How to debug locally# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip # view trace |



Summary
Cherry-picks three existing Dependabot bumps from
mainto fix security findings missed by #27940 on 1.13.org.eclipse.jetty:jetty-httpin/openmetadata-service12.1.6 → 12.1.7 (cherry-pick of Chore(deps): Bump org.eclipse.jetty:jetty-http from 12.1.6 to 12.1.7 in /openmetadata-service #27372)org.eclipse.jetty:jetty-httpin/openmetadata-mcp12.1.6 → 12.1.7 (cherry-pick of Chore(deps): Bump org.eclipse.jetty:jetty-http from 12.1.6 to 12.1.7 in /openmetadata-mcp #27373)org.pac4j:pac4j-corein/openmetadata-service5.7.0 → 5.7.10 (cherry-pick of Chore(deps): Bump org.pac4j:pac4j-core from 5.7.0 to 5.7.10 in /openmetadata-service #27503)Vulnerabilities resolved
org.eclipse.jetty:jetty-http12.1.6org.pac4j:pac4j-core5.7.0String.hashCode()allowed a forged token whose hash collides with the legitimate CSRF token to perform unauthorized state-changing actions (profile updates, password changes, account linking). Fixed in 5.7.10 / 6.4.1.Why these were missed
#27940 (commit
4c1ec72on 1.13) bumped<jetty.version>only in the rootpom.xml. Onmainthat was sufficient because three child-module Dependabot PRs had already landed:/openmetadata-service/openmetadata-mcp/openmetadata-serviceNone of those were cherry-picked to 1.13, so the child modules still redeclared the older versions as local properties:
openmetadata-service/pom.xml:24—<pac4j.version>5.7.0</pac4j.version>openmetadata-service/pom.xml:29—<jetty.version>12.1.6</jetty.version>openmetadata-mcp/pom.xml:20—<jetty.version>12.1.6</jetty.version>Maven property resolution gives the child module's local property precedence over the parent's, so every
jetty-*artifact pulled in byopenmetadata-serviceoropenmetadata-mcpwas still resolving to 12.1.6 at build time, and pac4j-core was still resolving to 5.7.0. The published JARs in/opt/openmetadata/libs/therefore still contained the vulnerable versions, and Snyk's May 8 scan against the 1.13 image correctly flagged them.Remaining gaps on 1.13 (out of scope for this PR — separate follow-ups)
io.netty:netty-bom@4.1.132.Final(4 findings — data amplification ×2, poison null byte, allocation throttling). Needs to land onmainfirst since main is also still on 4.1.132.Final. Tracked in fix(security): bump netty-bom 4.1.132.Final → 4.1.133.Final (4 HIGH CVEs) #27981.org.apache.logging.log4j:log4j-core@2.25.3(3 findings — Rfc5424Layout, XmlLayout, Log4j1XmlLayout). No upstream release available yet — Apache has only fixed in master. Recommend a Snyk policy ignore until Apache cuts a 2.25.4+ release.ReflectionUtil.createConnectionConfigClass.Test plan
mvn dependency:tree | grep jetty-httpresolves to 12.1.7 in bothopenmetadata-serviceandopenmetadata-mcpmvn dependency:tree | grep pac4j-coreresolves to 5.7.10 inopenmetadata-service🤖 Generated with Claude Code