As described in https://jira.apache.org/jira/browse/JCLOUDS-1447, S3 CopyObject 
requires the `x-amz-copy-source` header to be URL encoded.

This is not universally true of all headers though (for example the equals in 
`x-amz-copy-source-range`) therefore introducing a new parameter on `@Headers` 
to indicate whether URL encoding should take place.

Also updated `testUseBucketWithUpperCaseName` to remove the testing for legacy 
naming conventions as these are no longer supported by AWS.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Correct failing testUseBucketWithUpperCaseName
  * URL encode x-amz-copy-source for S3 CopyObject

-- File Changes --

    M apis/s3/src/main/java/org/jclouds/s3/S3Client.java (4)
    M apis/s3/src/test/java/org/jclouds/s3/S3ClientLiveTest.java (17)
    M apis/s3/src/test/java/org/jclouds/s3/S3ClientMockTest.java (15)
    M core/src/main/java/org/jclouds/rest/annotations/Headers.java (6)
    M core/src/main/java/org/jclouds/rest/internal/RestAnnotationProcessor.java 
(2)
    M 
providers/aws-s3/src/test/java/org/jclouds/aws/s3/AWSS3ClientLiveTest.java (46)

-- Patch Links --

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

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1238

Reply via email to