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.

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

Default servlet drops first accept-encoding header if there is more than one. #7863

Closed
markslater opened this issue Apr 8, 2022 · 1 comment · Fixed by #7864 or #7871
Closed

Default servlet drops first accept-encoding header if there is more than one. #7863

markslater opened this issue Apr 8, 2022 · 1 comment · Fixed by #7864 or #7871
Labels
Bug For general bugs on Jetty side

Comments

@markslater
Copy link
Contributor

Jetty version(s)
11.0.9

Java version/vendor (use: java -version)
N/A

OS type/version
N/A

Description
When DefaultServlet handles a request with multiple accept-encoding headers, it attempts to combine their values, but it drops the value of the first header (and produces a value with a leading comma).

How to reproduce?
Test case to follow.

@markslater markslater added the Bug For general bugs on Jetty side label Apr 8, 2022
markslater added a commit to markslater/jetty.project that referenced this issue Apr 8, 2022
…le accept-encoding headers are given.

Signed-off-by: markslater <[email protected]>
markslater added a commit to markslater/jetty.project that referenced this issue Apr 10, 2022
…le accept-encoding headers are given.

Signed-off-by: markslater <[email protected]>
joakime pushed a commit that referenced this issue Apr 11, 2022
…cept-encoding headers are given. (#7864)

Signed-off-by: markslater <[email protected]>
joakime pushed a commit that referenced this issue Apr 11, 2022
…cept-encoding headers are given. (#7864)

Signed-off-by: markslater <[email protected]>
joakime added a commit that referenced this issue Apr 27, 2022
…cept-encoding headers are given. (#7864) (#7871)

Signed-off-by: markslater <[email protected]>

Co-authored-by: markslater <[email protected]>
@joakime
Copy link
Contributor

joakime commented May 11, 2022

All merged into jetty-9.4.x and jetty-10.0.x+

@joakime joakime closed this as completed May 11, 2022
joakime pushed a commit that referenced this issue Oct 6, 2022
…cept-encoding headers are given. (#7864)

Signed-off-by: markslater <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For general bugs on Jetty side
Projects
None yet
2 participants