Skip to content

fix(security): bump netty-bom 4.1.132.Final → 4.1.133.Final (4 HIGH CVEs)#27981

Open
sonika-shah wants to merge 2 commits intomainfrom
fix/security-netty-bom-4.1.133
Open

fix(security): bump netty-bom 4.1.132.Final → 4.1.133.Final (4 HIGH CVEs)#27981
sonika-shah wants to merge 2 commits intomainfrom
fix/security-netty-bom-4.1.133

Conversation

@sonika-shah
Copy link
Copy Markdown
Collaborator

@sonika-shah sonika-shah commented May 8, 2026

Summary

Bumps io.netty:netty-bom from 4.1.132.Final → 4.1.133.Final to resolve four HIGH-severity Snyk findings observed in the May 8 2026 scan of the 1.13 image. A single bump on the netty-bom import propagates the version to every netty artifact pulled in transitively (codec, codec-http2, codec-dns, etc.), so this one-line change retires all four CVEs.

Vulnerabilities resolved

Severity Module CWE / Issue
HIGH io.netty:netty-codec-http2 Improper Handling of Highly Compressed Data — HttpContentDecompressor / DelegatingDecompressorFrameListener accept br/zstd/snappy payloads that bypass the configured decompression limit (data amplification → DoS)
HIGH io.netty:netty-codec-dns Null Byte Interaction Error — encodeDomainName / decodeDomainName inadequately validate domain-label lengths, enabling DNS cache poisoning, domain validation bypass, and excessive memory allocation
HIGH io.netty:netty-codec Allocation of Resources Without Limits or Throttling — Lz4FrameDecoder over-allocates on crafted compressed frames with manipulated header fields (resource exhaustion → DoS)
HIGH io.netty:netty-codec Improper Handling of Highly Compressed Data — same data-amplification class as above, in the base codec

All four are fixed in 4.1.133.Final (Snyk-recommended remediation: "Upgrade io.netty:netty-* to 4.1.133.Final or higher").

Why this is needed on main

The May 8 Snyk report against the 1.13 OSS image flagged 9 vulnerable dependency paths. The previous security cherry-pick (#27940 → 1.13 commit 4c1ec72) covered jetty-http, BouncyCastle, postgresql, jackson, spring, gson, httpcore5-h2 and commons-compress — but did not touch netty. main is also still on 4.1.132.Final, so this gap exists here as well; the bump has to land on main first before it can be cherry-picked into 1.13.x.

Other findings from the same Snyk report (tracked separately, not in this PR)

Test plan

  • mvn dependency:tree resolves io.netty:netty-codec, io.netty:netty-codec-http2, and io.netty:netty-codec-dns to 4.1.133.Final in openmetadata-service and openmetadata-mcp
  • Full backend build passes (mvn clean package -DskipTests)
  • Re-run Snyk scan against a fresh build and confirm all four netty findings are gone
  • Smoke test: HTTP/2 traffic, DNS resolution, and any Lz4-compressed paths still function

🤖 Generated with Claude Code


Summary by Gitar

  • UI/UX improvements:
    • Migrated ColumnProfileTable from antd to core components Table.

This will update automatically on new commits.

Resolves 4 HIGH-severity Snyk findings (May 8 2026 scan against 1.13 image):

- io.netty:netty-codec-http2 — Improper Handling of Highly Compressed Data
  (Data Amplification, br/zstd/snappy)
- io.netty:netty-codec-dns — Null Byte Interaction Error (Poison Null Byte)
- io.netty:netty-codec — Allocation of Resources Without Limits or Throttling
  (Lz4FrameDecoder)
- io.netty:netty-codec — Improper Handling of Highly Compressed Data
  (Data Amplification)

All four are fixed in netty 4.1.133.Final per the upstream advisories.
A single bump on the netty-bom import propagates the version to every
netty artifact pulled in transitively.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 8, 2026 07:03
@github-actions github-actions Bot added backend safe to test Add this label to run secure Github workflows on PRs labels May 8, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the root Maven dependency management to import io.netty:netty-bom 4.1.133.Final, ensuring all Netty artifacts resolve to the patched version across the build to address reported HIGH-severity CVEs.

Changes:

  • Bump io.netty:netty-bom from 4.1.132.Final → 4.1.133.Final in the root pom.xml dependency management import.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

🔴 Playwright Results — 1 failure(s), 13 flaky

✅ 4014 passed · ❌ 1 failed · 🟡 13 flaky · ⏭️ 86 skipped

Shard Passed Failed Flaky Skipped
🟡 Shard 1 297 0 2 4
🟡 Shard 2 750 0 5 8
🔴 Shard 3 757 1 1 7
🟡 Shard 4 787 0 3 18
✅ Shard 5 687 0 0 41
🟡 Shard 6 736 0 2 8

Genuine Failures (failed on all attempts)

Flow/ObservabilityAlerts.spec.ts › Alert operations for a user with and without permissions (shard 3)
�[31mTest timeout of 60000ms exceeded.�[39m
🟡 13 flaky test(s) (passed on retry)
  • Pages/AuditLogs.spec.ts › should apply both User and EntityType filters simultaneously (shard 1, 1 retry)
  • Pages/UserCreationWithPersona.spec.ts › Create user with persona and verify on profile (shard 1, 1 retry)
  • Features/ActivityAPI.spec.ts › Activity event shows the actor who made the change (shard 2, 1 retry)
  • Features/BulkEditEntity.spec.ts › Glossary (shard 2, 1 retry)
  • Features/DataProductRenameConsolidation.spec.ts › Multiple rename + update cycles - assets should be preserved (shard 2, 1 retry)
  • Features/Glossary/GlossaryWorkflow.spec.ts › should display correct status badge color and icon (shard 2, 1 retry)
  • Features/IncidentManager.spec.ts › Next, Previous and page indicator (shard 2, 1 retry)
  • Features/RTL.spec.ts › Verify Following widget functionality (shard 3, 1 retry)
  • Pages/CustomProperties.spec.ts › Time (shard 4, 1 retry)
  • Pages/DataContracts.spec.ts › Add and update Security and SLA tabs (shard 4, 1 retry)
  • Pages/DomainDataProductsRightPanel.spec.ts › Should edit owners for data product from domain context (shard 4, 1 retry)
  • Pages/Lineage/LineageFilters.spec.ts › Verify lineage schema filter selection (shard 6, 1 retry)
  • Pages/Users.spec.ts › Create and Delete user (shard 6, 1 retry)

📦 Download artifacts

How to debug locally
# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip    # view trace

@gitar-bot
Copy link
Copy Markdown

gitar-bot Bot commented May 8, 2026

Code Review ✅ Approved

Bumps netty-bom to 4.1.133.Final to address four high-severity vulnerabilities in netty-codec, netty-codec-http2, and netty-codec-dns. No issues found.

Options

Display: compact → Showing less information.

Comment with these commands to change:

Compact
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 8, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend safe to test Add this label to run secure Github workflows on PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants