Skip to content

bugfix: set expiration time for container credential provider#6481

Merged
zoewangg merged 4 commits intoaws:masterfrom
carhaz:bugfix/set-container-credentials-provider-expiration-time
Feb 6, 2026
Merged

bugfix: set expiration time for container credential provider#6481
zoewangg merged 4 commits intoaws:masterfrom
carhaz:bugfix/set-container-credentials-provider-expiration-time

Conversation

@carhaz
Copy link
Contributor

@carhaz carhaz commented Oct 15, 2025

Bugfix to set AwsSessionCredential expiration time so expiration time is retrievable for ContainerCredentialsProvider. Current clients cannot get the expiration time for session credentials in ECS containers using the java sdk.

Motivation and Context

Bugfix to set AwsSessionCredential expiration time so expiration time is retrievable for ContainerCredentialsProvider. Current clients cannot get the expiration time for session credentials in ECS containers using the java sdk.

Modifications

Adding line to builder in HttpCredentialsLoader.java for the builder for the AwsSessionCredentials.

Testing

mvn test -pl :auth

Ran targeted test because install failed on netty test before I even made changes.

Screenshots (if appropriate)

Types of changes

  • [ x] Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist

  • [ x] I have read the CONTRIBUTING document
  • Local run of mvn install succeeds
  • My code follows the code style of this project
  • My change requires a change to the Javadoc documentation
  • I have updated the Javadoc documentation accordingly
  • [x ] I have added tests to cover my changes
  • [x ] All new and existing tests passed
  • I have added a changelog entry. Adding a new entry must be accomplished by running the scripts/new-change script and following the instructions. Commit the new file created by the script in .changes/next-release with your changes.
  • My change is to implement 1.11 parity feature and I have updated LaunchChangelog

License

  • [x ] I confirm that this pull request can be released under the Apache 2 license

@carhaz carhaz requested a review from a team as a code owner October 15, 2025 22:37
@carhaz
Copy link
Contributor Author

carhaz commented Dec 6, 2025

Any update on getting a reviewer assigned?

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 2, 2026

@zoewangg zoewangg added the no-api-surface-area-change Indicate there is no API surface area change and thus API surface area review is not required label Feb 6, 2026
@zoewangg zoewangg enabled auto-merge February 6, 2026 04:04
@zoewangg zoewangg added this pull request to the merge queue Feb 6, 2026
Merged via the queue into aws:master with commit 5556232 Feb 6, 2026
22 of 46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-api-surface-area-change Indicate there is no API surface area change and thus API surface area review is not required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants