> @@ -133,6 +133,12 @@ HttpRequest replaceSecurityTokenHeader(HttpRequest 
> request, SessionCredentials c
>     }
>  
>     HttpRequest replaceAuthorizationHeader(HttpRequest request, String 
> signature) {
> +      // Only add the Authorization header if the query string doesn't 
> already contain
> +      // the 'Signature' parameter, otherwise S3 will fail the request 
> complaining about
> +      // duplicate authentication methods.
> +      if (request.getEndpoint().toString().contains("Signature")) {

Ah, then perhaps add to the comment that Signature will be present for 
temp-signed URLs?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/71/files#r5348973

Reply via email to