Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore(credential-providers): add credential attribution #6546

Merged
merged 8 commits into from
Oct 8, 2024

Conversation

kuhe
Copy link
Contributor

@kuhe kuhe commented Oct 4, 2024

Issue

n/a

Description

Adds feature detection for credential sources, with the exceptions of:

  • CREDENTIALS_STS_ASSUME_ROLE_SAML
  • CREDENTIALS_STS_FEDERATION_TOKEN
  • CREDENTIALS_STS_SESSION_TOKEN

which will be completed later due to needing further customization.

Testing

  • write new integ tests

@kuhe kuhe requested review from a team as code owners October 4, 2024 20:56
@kuhe kuhe marked this pull request as draft October 4, 2024 20:56
@kuhe kuhe marked this pull request as ready for review October 7, 2024 20:00
@kuhe kuhe force-pushed the chore/features-credentials branch from ce8eb61 to fea98a1 Compare October 7, 2024 20:13
@kuhe kuhe force-pushed the chore/features-credentials branch from fea98a1 to 834aa7a Compare October 7, 2024 20:13
*
* @returns the credentials with source feature attribution.
*/
export function setCredentialFeature<F extends keyof AwsSdkCredentialsFeatures>(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: move to a new file setCredentialFeature?

Copy link
Contributor Author

@kuhe kuhe Oct 8, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done, added unit tests

@kuhe kuhe merged commit 089f1a4 into aws:main Oct 8, 2024
5 checks passed
@kuhe kuhe deleted the chore/features-credentials branch October 8, 2024 16:20
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 23, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants