Skip to content

Adds accountId support to process credentials provider #4332

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

Conversation

cenedhryn
Copy link
Contributor

@cenedhryn cenedhryn commented Aug 22, 2023

Motivation and Context

Adds accountId support to process credentials provider

Modifications

  • Process credentials provider attempts to read accountId from the JSON file and add to credentials
  • Adds expiration time to the credentials as well since it's now supported
  • Modifies the test scripts that produce different JSON files so they can handle an arbitrary number of optional arguments, with expiration, session token and accountId being named.

@cenedhryn cenedhryn requested a review from a team as a code owner August 22, 2023 20:46
@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug B 1 Bug
Vulnerability E 1 Vulnerability
Security Hotspot A 0 Security Hotspots
Code Smell A 248 Code Smells

0.0% 0.0% Coverage
3.1% 3.1% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@cenedhryn cenedhryn merged commit ec2bfca into feature/master/accountid-endpoint-routing Aug 23, 2023
@cenedhryn cenedhryn deleted the salande/accountid-process-credentials-provider branch August 23, 2023 21:39
cenedhryn added a commit that referenced this pull request Apr 17, 2024
* Adds accountId support to process credentials provider

* Updates codegen tests from upstream branch to take account ID into account
cenedhryn added a commit that referenced this pull request Apr 22, 2024
…5119)

* Adds account ID as a built-in endpoint parameter (#4016)

* Refactors credential identity with separate implementing classes (#4024) (#4028)

* Adds accountId as a parameter to AWS credentials identity (#4029)

* Adds accountId to STS credentials providers (#4040)

* Adding accountId support to environment variable credential provider (#4327)

* Adds accountId support to process credentials provider (#4332)

* Adds account ID support for profile credentials provider sources (#4340)

* Adds accountId endpoint mode (#4984)

* Adding back existing endpoint files (#5120)

* changelog (#5121)
akidambisrinivasan pushed a commit to akidambisrinivasan/aws-sdk-java-v2 that referenced this pull request Jun 28, 2024
…ws#5119)

* Adds account ID as a built-in endpoint parameter (aws#4016)

* Refactors credential identity with separate implementing classes (aws#4024) (aws#4028)

* Adds accountId as a parameter to AWS credentials identity (aws#4029)

* Adds accountId to STS credentials providers (aws#4040)

* Adding accountId support to environment variable credential provider (aws#4327)

* Adds accountId support to process credentials provider (aws#4332)

* Adds account ID support for profile credentials provider sources (aws#4340)

* Adds accountId endpoint mode (aws#4984)

* Adding back existing endpoint files (aws#5120)

* changelog (aws#5121)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants