Repository navigation
⬆️ deps:(deps): bump io.netty:netty-transport-native-epoll from 4.2.17.Final to 4.2.18.Final - #658
Conversation
Bumps [io.netty:netty-transport-native-epoll](https://github.com/netty/netty) from 4.2.17.Final to 4.2.18.Final. - [Release notes](https://github.com/netty/netty/releases) - [Commits](netty/netty@netty-4.2.17.Final...netty-4.2.18.Final) --- updated-dependencies: - dependency-name: io.netty:netty-transport-native-epoll dependency-version: 4.2.18.Final dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
|
@claude Please review this dependency update against current dev for substantive compatibility, security and regression issues. Check version constraints, Kotlin/Compose alignment, native classifiers where applicable, and CI coverage. |
|
Claude finished @kshivang's task in 3m 30s —— View job Review: netty 4.2.17.Final → 4.2.18.FinalVerdict: safe to merge — but be aware it fixes none of the CVEs its release notes advertise. No compatibility or regression risk found in the code that consumes these artifacts. Two follow-ups worth opening separately.
What the diff actually doesAll three netty coordinates moved together, which is the part that matters: -netty-transport-native-epoll = { ... version = "4.2.17.Final" }
-netty-transport-native-kqueue = { ... version = "4.2.17.Final" }
-netty-transport-native-unix-common = { ... version = "4.2.17.Final" }
+ ... "4.2.18.Final" (all three)
But it is accidental, not enforced. Dependabot's metadata names only
1. Security: this bump carries none of the listed CVE fixes 🔴 (informational, not a blocker)Every advisory in the release notes is in a codec or handler artifact: The repo does consume the vulnerable artifacts, transitively:
Neither is moved by this PR, and there is no netty BOM and no dependency constraints anywhere in the build — 2. Split-version netty classpath 🟡 (pre-existing, widened by one patch)Because Gradle resolves highest-wins per module, That Worth confirming on a runner with network (I could not — no Gradle cache in this sandbox and network resolution is unavailable here): ./gradlew :boss-ipc:dependencies --configuration runtimeClasspath | grep io.nettyIf implementation(platform("io.netty:netty-bom:4.2.18.Final"))Caveat if you take the BOM: that is exactly when 4.2.18's "HTTP/2 header value validation is now enabled by default" would start applying. I checked the blast radius — BOSS IPC defines one custom header, 3. CI never exercises the code path this PR changes 🟡The whole reason these three artifacts exist is Unix domain sockets — and no test in the repo opens one. Every
Nothing constructs a That is the substantive regression exposure on any netty native bump, not just this one. A single 4. Native classifiers ✅
(Longer-term, 5. Kotlin / Compose alignment ✅No interaction. netty is a plain JVM runtime dependency of 6. Deprecation ✅
Summary
I did not run Gradle — dependency resolution needs network, which is unavailable in this sandbox, so finding #2 is stated as a verification step with the exact command rather than as a confirmed resolved-version claim. No files changed. |
|
@dependabot Thank you for the complete native transport update. Verified that original #658 (bb6283c) and #657 (747497b) have identical Git trees and the same three-module patch, including epoll, kqueue and unix-common. The older replacement #657 preserves explicit Dependabot credit for both PRs in its description. Your current Claude review is retained as input to that review; its runtime-alignment and native-test findings are being addressed there. Closing only this verified duplicate, not a distinct platform update. |
|
OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting If you change your mind, just re-open this PR and I'll resolve any conflicts on it. |
Bumps io.netty:netty-transport-native-epoll from 4.2.17.Final to 4.2.18.Final.
Release notes
Sourced from io.netty:netty-transport-native-epoll's releases.
... (truncated)
Commits
2521f49[maven-release-plugin] prepare release netty-4.2.18.Final6fd5327HTTP/1 absolute-form Host mismatch is translated to HTTP/3 :authority, overri...c44a052SPDY: SpdySessionHandler must limit the concurrent streams374d965HTTP: Limit the maximum number of concurrent pipelined requests7e8b325HTTP/2: Limit HPACK encoding table sizee3ebf70OCSP: Correctly handle thatnextUpdateis optional5388535STOMP: Correctly release partial content on handler removal3a80f5aWebSockets: Enforce a limit for the max pipelined requests in WebSocketServer...1b6ea48HTTP3: Correctly handle ":authority" and "host" headers3630659STOMP codec content-length long-to-int truncation causes infinite decode loop...Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)