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

Multiple values of a HTTP header are reduced to one value. #1071

Merged
merged 2 commits into from Nov 8, 2018

Conversation

Projects
None yet
3 participants
@volsch
Contributor

volsch commented Sep 6, 2018

If a HTTP response header has multiple values, these are reduced by class RestfulResponse to the last value (a non multi value map is used to collect the values). The class also returns this non multi value map so that the corresponding HttpServletResponse or JAX-RS Response is only populated with this value. The patch solves this issue and provides all header values.

@coveralls

This comment has been minimized.

coveralls commented Sep 6, 2018

Coverage Status

Coverage increased (+0.03%) to 74.581% when pulling 35c440b on volsch:master into 979b128 on jamesagnew:master.

@jamesagnew

This comment has been minimized.

Owner

jamesagnew commented Nov 8, 2018

Thanks Volker, looks great!

@jamesagnew jamesagnew merged commit 17a7fd4 into jamesagnew:master Nov 8, 2018

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.03%) to 74.581%
Details

jamesagnew added a commit that referenced this pull request Nov 8, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment