Switch to case insensitive comparison of HTTP headers#534
Switch to case insensitive comparison of HTTP headers#534jonsten wants to merge 1 commit intojfrog:masterfrom
Conversation
A bug was introduced in 67cefb2 where http headers no longer was compared case insensitively. This causes problems for Artifactory instances that run behind proxys that normalizes the http headers. This change fixes the issue by mimicking the implementation in org.apache.http.message.HeaderGroup by using String#equalsIgnoreCase(). Which is the old and correct behaviour.
|
CLA Assistant Lite bot: I have read the CLA Document and I hereby sign the CLA Jon Sten seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. |
|
Looks like we don't have any CLA signed. It will probably take a few weeks/months to get that setup through legal considering that it is vacation time right now... |
|
@jonsten, |
|
That is fine with me, thanks! |
Fixes #533
A bug was introduced in 67cefb2 where http headers no longer was compared case insensitively. This causes problems for Artifactory instances that run behind proxys that normalizes the http headers.
This change fixes the issue by mimicking the implementation in org.apache.http.message.HeaderGroup by using String#equalsIgnoreCase(). Which is the old and correct behaviour.
I haven't added any tests nor tried to build the code base since Gradle is unable to find some dependencies :(