fix: Credentials Resolution Bug in S3 Reads (#6444)

deephaven · Nov 27, 2024 · e7f27cb · e7f27cb
1 parent 0be4153
commit e7f27cb
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/extensions/s3/src/main/java/io/deephaven/extensions/s3/ResolvingCredentials.java b/extensions/s3/src/main/java/io/deephaven/extensions/s3/ResolvingCredentials.java
@@ -26,20 +26,18 @@
 enum ResolvingCredentials implements AwsSdkV2Credentials {
     INSTANCE;
 
-    private static final AwsCredentialsProviderChain PROVIDER_CHAIN = AwsCredentialsProviderChain.builder()
-            .credentialsProviders(
-                    DefaultCredentialsProvider.create(),
-                    AnonymousCredentialsProvider.create())
-            .reuseLastProviderEnabled(false) // Don't cache because this chain is a shared static instance
-            .build();
-
     @Override
     public final AwsCredentialsProvider awsV2CredentialsProvider(@NotNull final S3Instructions instructions) {
         if (instructions.profileName().isPresent()
                 || instructions.configFilePath().isPresent()
                 || instructions.credentialsFilePath().isPresent()) {
             return ProfileCredentials.INSTANCE.awsV2CredentialsProvider(instructions);
         }
-        return PROVIDER_CHAIN;
+        return AwsCredentialsProviderChain.builder()
+                .credentialsProviders(
+                        DefaultCredentialsProvider.create(),
+                        AnonymousCredentialsProvider.create())
+                .reuseLastProviderEnabled(true)
+                .build();
     }
 }