specifying a region for S3Client

Author: ORybak5

service/src/main/java/uk/nhs/adaptors/gp2gp/common/storage/StorageConnectorConfiguration.java

@@ -5,6 +5,7 @@
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
 
+import software.amazon.awssdk.regions.Region;
 import software.amazon.awssdk.services.s3.S3Client;
 
 import lombok.Getter;
@@ -26,7 +27,7 @@ public class StorageConnectorConfiguration {
     @Bean
     public S3Client getS3Client() {
         if (StringUtils.isNotBlank(trustStoreUrl) && trustStoreUrl.startsWith(S3_PREFIX)) {
-            return S3Client.builder().build();
+            return S3Client.builder().region(Region.EU_WEST_2).build();
         }
 
         return null;

service/src/test/java/uk/nhs/adaptors/gp2gp/common/storage/S3StorageConnectorTest.java

@@ -6,6 +6,7 @@
 import org.mockito.MockitoAnnotations;
 import software.amazon.awssdk.core.ResponseInputStream;
 import software.amazon.awssdk.core.sync.RequestBody;
+import software.amazon.awssdk.regions.Region;
 import software.amazon.awssdk.services.s3.S3Client;
 import software.amazon.awssdk.services.s3.model.GetObjectRequest;
 import software.amazon.awssdk.services.s3.model.GetObjectResponse;
@@ -49,8 +50,8 @@ void setUp() {
 
 
     @Test
-    void expectExceptionWhenS3ClientCantDeliverResponse() {
-        S3StorageConnector storageConnector = new S3StorageConnector(S3Client.builder().build(), config);
+    void expectExceptionWhenS3ClientCannotDeliverResponse() {
+        S3StorageConnector storageConnector = new S3StorageConnector(S3Client.builder().region(Region.EU_WEST_2).build(), config);
         Exception exception = assertThrows(StorageConnectorException.class,
                                            () -> storageConnector.downloadFromStorage("s3File"));