Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve buffer management of HTTP/1 response headers #12670

Open
sbordet opened this issue Jan 4, 2025 · 0 comments
Open

Improve buffer management of HTTP/1 response headers #12670

sbordet opened this issue Jan 4, 2025 · 0 comments
Assignees
@sbordet
Labels
Bug For general bugs on Jetty side Sponsored This issue affects a user with a commercial support agreement

Comments

@sbordet
Copy link
Contributor

sbordet commented Jan 4, 2025

Jetty version(s)
12.0.x

Description
After #12351, the server-side allocates the response headers using HttpConfiguration.responseHeaderSize.

However, #12544 introduced a more dynamic mechanism on the client-side where the generation of response headers is first tried with responseHeaderSize and in case of overflow, re-tried with maxResponseHeaderSize.

A similar mechanism should be added to the server-side.
@sbordet sbordet added the Bug For general bugs on Jetty side label Jan 4, 2025
@sbordet sbordet self-assigned this Jan 4, 2025
@sbordet sbordet added the Sponsored This issue affects a user with a commercial support agreement label Jan 4, 2025
wendigo added a commit to airlift/airlift that referenced this issue Jan 5, 2025
@wendigo
Configure Buffer pools for Jetty
bf95d76 
This circumvents jetty/jetty.project#12670
@wendigo wendigo mentioned this issue Jan 5, 2025
Closed
3 tasks
wendigo added a commit to airlift/airlift that referenced this issue Jan 5, 2025
@wendigo
Configure Buffer pools for Jetty
0ef5ccf 
This circumvents jetty/jetty.project#12670
@wendigo wendigo mentioned this issue Jan 6, 2025
Closed
wendigo added a commit to airlift/airlift that referenced this issue Jan 7, 2025
@wendigo
Configure Buffer pools for Jetty
25abb54 
This circumvents jetty/jetty.project#12670
wendigo added a commit to wendigo/airlift that referenced this issue Jan 9, 2025
@wendigo
Configure Buffer pools for Jetty server
5c3e7d1 
This circumvents jetty/jetty.project#12670
wendigo added a commit to airlift/airlift that referenced this issue Jan 13, 2025
@wendigo
Configure Buffer pools for Jetty server
bde22d9 
This circumvents jetty/jetty.project#12670
wendigo added a commit to airlift/airlift that referenced this issue Jan 14, 2025
@wendigo
Configure Buffer pools for Jetty server
faf2c4e 
This circumvents jetty/jetty.project#12670
wendigo added a commit to airlift/airlift that referenced this issue Jan 14, 2025
@wendigo
Configure Buffer pools for Jetty server
04b9fd8 
This circumvents jetty/jetty.project#12670
wendigo added a commit to wendigo/airlift that referenced this issue Jan 14, 2025
@wendigo
Configure Buffer pools for Jetty server
5b8f0b6 
This circumvents jetty/jetty.project#12670
wendigo added a commit to airlift/airlift that referenced this issue Jan 14, 2025
@wendigo
Configure Buffer pools for Jetty server
dd99962 
This circumvents jetty/jetty.project#12670
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sbordet sbordet

Labels
Bug For general bugs on Jetty side Sponsored This issue affects a user with a commercial support agreement
Projects
Jetty 12.0.17
Status: No status
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sbordet