The problem reported in JCLOUDS-155 is that certain proxies lowercase
headers. This is especially a problem for the swift header
"X-Storage-Url", because it tells the client after authentication 
where
to go to next.
The patch originally put the value into a CaseInsensitiveMap, but it was
dropped not to add a dependency to apache commons.
When that was dropped the case insensitivity was still checked against,
but the map entries contained the original header name as key.

When then the Factory tried to read AuthHeaders.STORAGE_URL from the map,
that value was no longer there.

This patch fixes also the test by testing that the new URL can be read
using that constant.
You can merge this Pull Request by running:

  git pull https://github.com/CodingFabian/jclouds jclouds-155

Or you can view, comment on it, or merge it online at:

  https://github.com/jclouds/jclouds/pull/467

-- Commit Summary --

  * Fixes patch for JCLOUDS-155.

-- File Changes --

    M 
common/openstack/src/main/java/org/jclouds/openstack/functions/ParseAuthenticationResponseFromHeaders.java
 (12)
    M 
common/openstack/src/test/java/org/jclouds/openstack/functions/ParseAuthenticationResponseFromHeadersTest.java
 (7)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/467.patch
https://github.com/jclouds/jclouds/pull/467.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/467

Reply via email to