fix: added wrapped profile default supplier

awslabs · Jan 10, 2025 · 5b1d7de · 5b1d7de
1 parent a545a36
commit 5b1d7de
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/...a/software/amazon/event/kafkaconnector/auth/EventBridgeAwsCredentialsProviderFactory.java b/...a/software/amazon/event/kafkaconnector/auth/EventBridgeAwsCredentialsProviderFactory.java
@@ -83,8 +83,7 @@ public static AwsCredentialsProvider getAwsCredentialsProvider(EventBridgeSinkCo
     return getStsAssumeRoleCredentialsProvider(config);
   }
 
-  // Temporary workaround until AWS SDK issue is fixed:
-  // https://github.com/aws/aws-sdk-java-v2/issues/5635
+  // Temporary workaround until AWS SDK issue is fixed: https://github.com/aws/aws-sdk-java-v2/issues/5635
   private static ProfileFileSupplier wrappedProfileDefaultSupplier() {
     return () -> {
       try {

diff --git a/.../software/amazon/event/kafkaconnector/auth/EventBridgeCredentialsProviderFactoryTest.java b/.../software/amazon/event/kafkaconnector/auth/EventBridgeCredentialsProviderFactoryTest.java
@@ -67,8 +67,8 @@ public void shouldUseAwsCredentialsProviderByProvidedClass() {
     assertThat(provider).isExactlyInstanceOf(AwsCredentialProviderImpl.class);
   }
 
-  //Testing wrapper functionality as a quick fix until the SDK issue is solved: https://github.com/aws/aws-sdk-java-v2/issues/5635
   @Test
+  //Testing wrapper functionality as a quick fix until the SDK issue is solved: https://github.com/aws/aws-sdk-java-v2/issues/5635
   public void shouldNotGetSDKClientExceptionWithNull() {
     var props = new HashMap<>(commonProps);
     var provider = EventBridgeAwsCredentialsProviderFactory.getAwsCredentialsProvider(new EventBridgeSinkConfig(props));