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

EncodingFilter handles 'Accept-Encoding' header with empty String awkwardly #4068

Closed
tkroehli opened this issue Feb 22, 2019 · 1 comment
Closed
Labels
Milestone

Comments

@tkroehli
Copy link

The class org.glassfish.jersey.server.filter.EncodingFillter only checks if the list of values for the 'Accept-Encoding' header is empty but not if a value in the list is empty. A reader is used in ContentEncoding#fromString for this value and the method hasNext is called, but its result is unused. This leads to a ParseException when nextToken is called for an empty String.
The exception does not lead to errors as it is caught, but still it can lead to lots of unnecessary exceptions and logging output.

@senivam
Copy link
Contributor

senivam commented Feb 26, 2019

fixed

@senivam senivam closed this as completed Feb 26, 2019
@senivam senivam added this to the 2.29 milestone Sep 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants