Skip to content

Commit

Permalink
Allowing async refresh for role credential provider
Browse files Browse the repository at this point in the history
  • Loading branch information
Siddharth Yagnik committed Jul 10, 2024
1 parent df5d25d commit 67acb73
Showing 1 changed file with 6 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -166,8 +166,9 @@ protected AwsCredentialsProvider getDefaultProvider() {

@Override
public AwsCredentials resolveCredentials() {
log.error("RESOLVING CREDENTIALS!!");
AwsCredentials credentials = loadCredentialsWithRetry();
if (credentials != null && shouldDebugCreds && log.isDebugEnabled()) {
if (credentials != null && shouldDebugCreds) {
logCallerIdentity(credentials);
}
return credentials;
Expand Down Expand Up @@ -218,7 +219,7 @@ private void logCallerIdentity(AwsCredentials credentials) {
try {
StsClient stsClient = getStsClientForDebuggingCreds(credentials);
GetCallerIdentityResponse response = stsClient.getCallerIdentity();
log.debug("The identity of the credentials is {}", response.toString());
log.error("The identity of the credentials is {}", response.toString());
} catch (Exception e) {
//If we run into an exception logging the caller identity, we should log the exception but
//continue running.
Expand Down Expand Up @@ -362,6 +363,7 @@ StsAssumeRoleCredentialsProvider createSTSRoleCredentialProvider(
return StsAssumeRoleCredentialsProvider.builder()
.stsClient(stsClient)
.refreshRequest(roleRequest)
.asyncCredentialUpdateEnabled(true)
.build();
}

Expand All @@ -379,6 +381,7 @@ StsAssumeRoleCredentialsProvider createSTSRoleCredentialProvider(
return StsAssumeRoleCredentialsProvider.builder()
.stsClient(stsClient)
.refreshRequest(roleRequest)
.asyncCredentialUpdateEnabled(true)
.build();
}

Expand All @@ -395,6 +398,7 @@ StsAssumeRoleCredentialsProvider createSTSRoleCredentialProvider(
return StsAssumeRoleCredentialsProvider.builder()
.stsClient(getStsClientBuilder(Region.of(stsRegion)).build())
.refreshRequest(roleRequest)
.asyncCredentialUpdateEnabled(true)
.build();
}
}
Expand Down

0 comments on commit 67acb73

Please sign in to comment.