S3 Bucket Location does not work when using Builder for client building #1142

Activity

1. kiiadi

   added

   feature-requestA feature should be added or improved.A feature should be added or improved.

   on May 2, 2017
2. kiiadi commented on May 2, 2017

   

   kiiadi

   on May 2, 2017

   Contributor

   More actions

   @margussipria there are a couple of work arounds you can use path-style addressing ie:

   AmazonS3 client = AmazonS3Client.builder().withRegion("us-east-1").withPathStyleAccessEnabled(true).build();
   client.getBucketLocation("your-eu-west-1-bucket");

   Alternatively you can catch the AmazonS3Exception and get the Region from the additionalDetails on the exception :

   try {
      client.getBucketLocation("your-eu-west-1-bucket");
   } catch (AmazonS3Exception e) {
      e.getAdditionalDetails().get("Region");
   }

   I agree neither are very pretty - we'll see if we can add one of these hooks to the SDK itself (need to be mindful of backwards compatibility).

   👍React with 👍2Reacted by Raniz Daniel Raneland and Gary Hewett
3. anishkej commented on May 9, 2017

   

   anishkej

   on May 9, 2017

   More actions

   I agree. A fix to make this seamless would be great.

   Thanks for the suggestions on how to work around this - we resorted to catching the exception, and getting the region from the message.
4. millems commented on May 9, 2017

   

   millems

   on May 9, 2017

   Contributor

   More actions

   One of the reasons this may be working on the old client (I'd have to see more detailed code samples to be sure) is because it may actually performing two separate calls. The first to the us-east-1 endpoint, and the second to eu-west-1 based on the exception message.

   If you're okay with the two calls and you just want the same behavior as the deprecated client constructor, you can try enabling global bucket access on the builder to see if your problem is fixed:

   AmazonS3 client = AmazonS3Client.builder().withRegion("us-east-1").withForceGlobalBucketAccessEnabled(true).build();
   client.getBucketLocation("your-eu-west-1-bucket");

   👍React with 👍1Reacted by Peter Groesbeck
5. shorea

   assigned

   dagnir

   on May 15, 2017
6. rjstanford commented on Jul 17, 2017

   

   rjstanford

   on Jul 17, 2017

   More actions

   client.getBucketLocation("your-eu-west-1-bucket");

   When I tried to use this approach, I kept getting "US" returned, which despite being in the enum and pointing to US_Standard isn't accepted as a Region when you try to set it on builder.withRegion(). Do you know of any other way to get a usable region back?

   Until then we're doing the double-call method described above.
7. jansyk13 commented on Mar 5, 2018

   

   jansyk13

   on Mar 5, 2018

   More actions

   @rjstanford @kiiadi IMO correct call would be client.headBucket(new HeadBucketRequest(bucket)).getBucketRegion()
   Btw it requires s3:ListBucket permission, see https://docs.aws.amazon.com/AmazonS3/latest/API/RESTBucketHEAD.html for more info.
8. debora-ito commented on Aug 21, 2019

   

   debora-ito

   on Aug 21, 2019

   Member

   More actions

   @margussipria @anishkej @rjstanford @jansyk13

   The SDK team has reviewed the feature request list for V1, and they decided to not implement this in V1 since they're concentrating efforts on V2 new features. One of the reasons is it would not be possible to change this without making breaking changes.

   Please feel free to reach out if you have further questions.
9. debora-ito

   closed this as completedon Aug 21, 2019