Skip to content

Commit edee66a

Browse files
committed
attempting to access more regions than are in the Regions enum
1 parent 6efea2e commit edee66a

File tree

2 files changed

+10
-6
lines changed

2 files changed

+10
-6
lines changed

src/main/java/usace/cc/plugin/CcStoreS3.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,8 @@
1717
import com.amazonaws.auth.BasicAWSCredentials;
1818
import com.amazonaws.client.builder.AwsClientBuilder;
1919
import com.amazonaws.regions.Regions;
20+
import com.amazonaws.regions.Region;
21+
import com.amazonaws.regions.RegionUtils;
2022
import com.amazonaws.services.s3.AmazonS3;
2123
import com.amazonaws.services.s3.AmazonS3ClientBuilder;
2224
import com.amazonaws.services.s3.model.AmazonS3Exception;
@@ -54,7 +56,7 @@ public CcStoreS3(){
5456
//System.out.println(EnvironmentVariables.CC_PROFILE + "_" + EnvironmentVariables.AWS_ACCESS_KEY_ID+"::"+config.aws_access_key_id);
5557
//System.out.println(EnvironmentVariables.CC_PROFILE + "_" + EnvironmentVariables.AWS_SECRET_ACCESS_KEY+"::"+config.aws_secret_access_key_id);
5658
//System.out.println(EnvironmentVariables.CC_PROFILE + "_" + EnvironmentVariables.AWS_S3_BUCKET+"::"+config.aws_bucket);
57-
Regions clientRegion = Regions.valueOf(config.aws_region.toUpperCase().replace("-", "_"));
59+
Region clientRegion = RegionUtils.getRegion(config.aws_region.toUpperCase().replace("-", "_"));//Regions.valueOf(config.aws_region.toUpperCase().replace("-", "_"));
5860
try {
5961
AmazonS3 s3Client = null;
6062
if(config.aws_mock){
@@ -70,7 +72,7 @@ public CcStoreS3(){
7072

7173
s3Client = AmazonS3ClientBuilder
7274
.standard()
73-
.withEndpointConfiguration(new AwsClientBuilder.EndpointConfiguration(config.aws_endpoint, clientRegion.name()))
75+
.withEndpointConfiguration(new AwsClientBuilder.EndpointConfiguration(config.aws_endpoint, clientRegion.getName()))
7476
.withPathStyleAccessEnabled(config.aws_force_path_style)
7577
.withClientConfiguration(clientConfiguration)
7678
.withCredentials(new AWSStaticCredentialsProvider(credentials))
@@ -79,7 +81,7 @@ public CcStoreS3(){
7981
AWSCredentials credentials = new BasicAWSCredentials(config.aws_access_key_id, config.aws_secret_access_key_id);
8082
s3Client = AmazonS3ClientBuilder
8183
.standard()
82-
.withRegion(clientRegion)
84+
.withRegion(clientRegion.getName())
8385
.withCredentials(new AWSStaticCredentialsProvider(credentials))
8486
.build();
8587
}

src/main/java/usace/cc/plugin/FileDataStoreS3.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,8 @@
1414
import com.amazonaws.auth.BasicAWSCredentials;
1515
import com.amazonaws.client.builder.AwsClientBuilder;
1616
import com.amazonaws.regions.Regions;
17+
import com.amazonaws.regions.Region;
18+
import com.amazonaws.regions.RegionUtils;
1719
import com.amazonaws.services.s3.AmazonS3;
1820
import com.amazonaws.services.s3.AmazonS3ClientBuilder;
1921
import com.amazonaws.services.s3.model.DeleteObjectRequest;
@@ -101,7 +103,7 @@ public FileDataStoreS3(DataStore ds){
101103
//System.out.println(ds.getDsProfile() + "_" + EnvironmentVariables.AWS_SECRET_ACCESS_KEY+"::"+config.aws_secret_access_key_id);
102104
//System.out.println(ds.getDsProfile() + "_" + EnvironmentVariables.AWS_S3_BUCKET+"::"+config.aws_bucket);
103105

104-
Regions clientRegion = Regions.valueOf(config.aws_region.toUpperCase().replace("-", "_"));
106+
Region clientRegion = RegionUtils.getRegion(config.aws_region.toUpperCase().replace("-", "_"));//Regions.valueOf(config.aws_region.toUpperCase().replace("-", "_"));
105107
try {
106108
AmazonS3 s3Client = null;
107109
if(config.aws_mock){
@@ -112,7 +114,7 @@ public FileDataStoreS3(DataStore ds){
112114

113115
s3Client = AmazonS3ClientBuilder
114116
.standard()
115-
.withEndpointConfiguration(new AwsClientBuilder.EndpointConfiguration(config.aws_endpoint, clientRegion.name()))
117+
.withEndpointConfiguration(new AwsClientBuilder.EndpointConfiguration(config.aws_endpoint, clientRegion.getName()))
116118
.withPathStyleAccessEnabled(config.aws_force_path_style)
117119
.withClientConfiguration(clientConfiguration)
118120
.withCredentials(new AWSStaticCredentialsProvider(credentials))
@@ -121,7 +123,7 @@ public FileDataStoreS3(DataStore ds){
121123
AWSCredentials credentials = new BasicAWSCredentials(config.aws_access_key_id, config.aws_secret_access_key_id);
122124
s3Client = AmazonS3ClientBuilder
123125
.standard()
124-
.withRegion(clientRegion)
126+
.withRegion(clientRegion.getName())
125127
.withCredentials(new AWSStaticCredentialsProvider(credentials))
126128
.build();
127129
}

0 commit comments

Comments
 (0)