HADOOP-17305. Fix ITestCustomSigner to work with s3 compatible endpoints (#2395)

mukund-thakur · web-flow · commit 7f8ef76c4833 · 2020-10-21T13:01:13.000+01:00
Contributed by Mukund Thakur
diff --git a/hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/auth/ITestCustomSigner.java b/hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/auth/ITestCustomSigner.java
@@ -133,11 +133,7 @@ private Configuration createTestConfig(String identifier) {
   }
 
   private String determineRegion(String bucketName) throws IOException {
-    AmazonS3 s3 = AmazonS3ClientBuilder.standard().withCredentials(
-        new SimpleAWSCredentialsProvider(null, createConfiguration()))
-        .withForceGlobalBucketAccessEnabled(true).withRegion("us-east-1")
-        .build();
-    String region = s3.getBucketLocation(bucketName);
+    String region = getFileSystem().getBucketLocation(bucketName);
     return fixBucketRegion(region);
   }
 

Original file line number	Diff line number	Diff line change
`@@ -133,11 +133,7 @@ private Configuration createTestConfig(String identifier) {`
`133`	`133`	`}`
`134`	`134`
`135`	`135`	`private String determineRegion(String bucketName) throws IOException {`
`136`		`- AmazonS3 s3 = AmazonS3ClientBuilder.standard().withCredentials(`
`137`		`- new SimpleAWSCredentialsProvider(null, createConfiguration()))`
`138`		`- .withForceGlobalBucketAccessEnabled(true).withRegion("us-east-1")`
`139`		`- .build();`
`140`		`- String region = s3.getBucketLocation(bucketName);`
	`136`	`+ String region = getFileSystem().getBucketLocation(bucketName);`
`141`	`137`	`return fixBucketRegion(region);`
`142`	`138`	`}`
`143`	`139`