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

Fix getResponseHeader(key) for IE11 #4173

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@Andrei15193

Andrei15193 commented Sep 11, 2018

Summary

This issue is described in more depth by #3403.

getResponseHeader(key) combines all header values for the provided key into a
single result where values are concatenated by ', '. This does not happen for
IE11 since multiple values for the same header are returned on separate lines.
This makes the function only return the last value of the header for IE11.

Checklist

Fix getResponseHeader(key) for IE11
getResponseHeader(key) combines all header values for the provided key into a
single result where values are concatenated by ', '. This does not happen for
IE11 since multiple values for the same header are returned on separate lines.
This makes the function only return the last value of the header for IE11.

Fixes #3403
@jsf-clabot

This comment has been minimized.

Show comment
Hide comment
@jsf-clabot

jsf-clabot Sep 11, 2018

CLA assistant check
All committers have signed the CLA.

jsf-clabot commented Sep 11, 2018

CLA assistant check
All committers have signed the CLA.

@mgol

This comment has been minimized.

Show comment
Hide comment
@mgol

mgol Oct 1, 2018

Member

Thanks for the PR. We need tests to land any fix like that.

Member

mgol commented Oct 1, 2018

Thanks for the PR. We need tests to land any fix like that.

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