Skip to content

Auto-port 4.2: Limit the number of Continuation frames per HTTP2 Headers#16536

Merged
normanmaurer merged 1 commit into4.2from
auto-port-pr-13969-to-4.2
Mar 24, 2026
Merged

Auto-port 4.2: Limit the number of Continuation frames per HTTP2 Headers#16536
normanmaurer merged 1 commit into4.2from
auto-port-pr-13969-to-4.2

Conversation

@netty-project-bot
Copy link
Copy Markdown
Contributor

Auto-port of #13969 to 4.2
Cherry-picked commit: 9f47a7b


Motivation:

We should limit the number of continuation frames that the remote peer is allowed to sent per headers.

Modifications:

  • Limit the number of continuation frames by default to 16 and allow the user to change this.
  • Add unit test

Result:

Do some more validations to guard against resource usage

Motivation:

We should limit the number of continuation frames that the remote peer
is allowed to sent per headers.

Modifications:

- Limit the number of continuation frames by default to 16 and allow the
user to change this.
- Add unit test

Result:

Do some more validations to guard against resource usage

---------

Co-authored-by: Bryce Anderson <bl_anderson@apple.com>
Co-authored-by: Chris Vest <mr.chrisvest@gmail.com>
(cherry picked from commit 9f47a7b)
@normanmaurer normanmaurer merged commit aae944a into 4.2 Mar 24, 2026
18 of 20 checks passed
@normanmaurer normanmaurer deleted the auto-port-pr-13969-to-4.2 branch March 24, 2026 14:53
@chrisvest chrisvest added this to the 4.2.11.Final milestone Mar 24, 2026
dongjoon-hyun added a commit to apache/spark-kubernetes-operator that referenced this pull request Mar 25, 2026
### What changes were proposed in this pull request?

This PR upgrades `Netty` to 4.2.12.Final.

### Why are the changes needed?

To bring in the latest bug fixes.
- https://netty.io/news/2026/03/24/4-2-12-Final.html
  - netty/netty#16550
- https://netty.io/news/2026/03/24/4-2-11-Final.html
  - netty/netty#16489
  - netty/netty#16536
  - netty/netty#16412

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

CI should pass with the existing tests.

### Was this patch authored or co-authored using generative AI tooling?

Generated-by: Claude Code (Claude Opus 4.6)

Closes #589 from dongjoon-hyun/SPARK-56213.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
dongjoon-hyun added a commit to apache/spark that referenced this pull request Mar 26, 2026
### What changes were proposed in this pull request?

This PR upgrades `Netty` to 4.2.12.Final.

### Why are the changes needed?

To bring in the latest bug fixes.
- https://netty.io/news/2026/03/24/4-2-12-Final.html
  - netty/netty#16550
- https://netty.io/news/2026/03/24/4-2-11-Final.html
  - netty/netty#16489
  - netty/netty#16536
  - netty/netty#16412

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

CI should pass with the existing tests.

### Was this patch authored or co-authored using generative AI tooling?

Generated-by: Claude Code (Claude Opus 4.6)

Closes #55016 from dongjoon-hyun/SPARK-56214.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
tappenzeller added a commit to Netflix/zuul that referenced this pull request Apr 2, 2026
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.

3 participants