-
Notifications
You must be signed in to change notification settings - Fork 863
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updated credential providers to provide Async version for generating credentials. #3681
base: v4-development
Are you sure you want to change the base?
Changes from 1 commit
685cb86
e5a45f5
93895bc
66a2926
33a2493
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
…credentials.
- Loading branch information
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
{ | ||
"core": { | ||
"changeLogMessages": [ | ||
"Updated credential providers to provide Async version for generating credentials." | ||
], | ||
"type": "patch", | ||
"updateMinimum": true | ||
}, | ||
"services": [ | ||
{ | ||
"serviceName": "SecurityToken", | ||
"type": "patch", | ||
"changeLogMessages": [ | ||
"Added implementation for ICoreAmazonSTS_SAML/ICoreAmazonSTS.CredentialsFromSAMLAuthenticationAsync() and ICoreAmazonSTS.CredentialsFromAssumeRoleAuthenticationAsync()." | ||
] | ||
} | ||
] | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -35,11 +35,9 @@ public abstract class AWSCredentials : BaseIdentity | |
/// </summary> | ||
protected virtual void Validate() { } | ||
|
||
#if AWS_ASYNC_API | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why are we removing There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I removed it based on discussion with @normj since it is now redundant. And based on discussion, the current scope of removal was limited to credential providers. |
||
public virtual System.Threading.Tasks.Task<ImmutableCredentials> GetCredentialsAsync() | ||
{ | ||
return System.Threading.Tasks.Task.FromResult<ImmutableCredentials>(this.GetCredentials()); | ||
} | ||
#endif | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make the change log something users will understand which don't directly interact with the
ICoreX
interfaces. Something like add async code paths for the SAML and Assume role credentials providers.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.