S3 support classes take Supplier<ProfileFile> #3653
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dave-fn
commented
Dec 22, 2022
Comment on lines
-84
to
+88
| this.useArnRegionEnabled = FieldWithDefault.createLazy(builder.useArnRegionEnabled, this::resolveUseArnRegionEnabled); | ||
| this.multiRegionEnabled = FieldWithDefault.createLazy(builder.multiRegionEnabled, this::resolveMultiRegionEnabled); | ||
| this.useArnRegionEnabled = builder.useArnRegionEnabled; | ||
| this.multiRegionEnabled = builder.multiRegionEnabled; |
Contributor
Author
There was a problem hiding this comment.
Changed so that these values are resolved per invocation, based on the active ProfileFile.
dagnir
reviewed
Jan 4, 2023
| /** | ||
| * The supplier of profile file instances that should be consulted to determine the default value of | ||
| * {@link #useArnRegionEnabled(Boolean)} or {@link #multiRegionEnabled(Boolean)}. | ||
| * This is not used, if those parameters are configured. |
Contributor
There was a problem hiding this comment.
nit: "configured on the builder"?
|
|
||
| System.clearProperty(AWS_CONFIG_FILE.property()); | ||
| String falseConfigFile = getClass().getResource("ProfileFile_false").getFile(); | ||
| System.setProperty(AWS_CONFIG_FILE.property(), falseConfigFile); |
Contributor
There was a problem hiding this comment.
minor: can you reset these in an @After method instead? Slightly safer to do in case this statement doesn't get reached during the test.
Contributor
Author
There was a problem hiding this comment.
Split test into two methods.
|
SonarCloud Quality Gate failed.
|
dagnir
approved these changes
Jan 6, 2023
dave-fn
added a commit
that referenced
this pull request
Feb 6, 2023
* ProfileCredentialsProvider can now reload credentials when profile files change (#3487) * ProfileFile can update if disk changed, reload as new instance * ProfileCredentialsProvider reloads credentials if profile file has changes * Created class ProfileFileRefresher * Created ReloadingProfileCredentialsProvider; moved new logic in ProfileFile to ProfileFileRefresher * Fix ReloadingProfileCredentialsBehavior when missing ProfileFile Supplier or Predicate, and dealing with defaults * Consolidated ReloadingProfileCredentialsProvider functionality into ProfileCredentialsProvider * Fix behavior when dealing with defaults * Created ProfileFileSupplier interface; refactored * Misc fixes * Created package-private ProfileFileSupplierBuilder; ProfileFileSupplier now extends Supplier; Fixed Javadoc * Fixed unit tests for default credentials file * Removed ProfileFileSupplier.Builder interface * Code cleanup * ProfileFileSupplier API changes, aggregating (#3558) * Added methods for aggregating ProfileFile objects * Removed redundant logic, changelog entry * Removed redundant methods * Use compare and set to make thread safe * DefaultCredentialsProvider reload profile (#3580) * Misc fixes * Reload credentials by DefaultCredentialsProvider; pass supplier to InstanceProfileCredentialsProvider * Fix code alignment * Update public APIs to Supplier<ProfileFile> instead of ProfileFileSupplier (#3607) * ProfileTokenProvider Reload ProfileFile (#3608) * Updated ProfileTokenProvider * Updated tests, do not explicitly swallow exceptions * ProfileTokenProviderLoader can use Supplier<Profilefile> internally * Simplified ProfileTokenProviderLoader API; implemented synchronized block * Use synchronized block (#3646) * S3 support classes take Supplier<ProfileFile> (#3653) * S3 support classes take Supplier<ProfileFile> * Review comments * Presigners, other Support classes take Supplier<ProfileFile> (#3677) * Presigners, other Support classes take Supplier<ProfileFile> * Split new ProfileFile tests from existing parameterized tests * Improved tests readability * ProfileFile updates to BaseClientBuilderClass, other S3 classes (#3685) * Leverage SdkClientOption and SdkExecutionAttributes; fallback to simp… (#3692) * Leverage SdkClientOption and SdkExecutionAttributes; fallback to simple ProfileFile * Addressed review comments * Updated changelog entry (#3699) * Fixed review comments * Removed unnecessary logic * Deprecated SdkClientOption PROFILE_FILE * Deprecated SdkExecutionAttribute PROFILE_FILE * Updated changelog entry
aws-sdk-java-automation
added a commit
that referenced
this pull request
Feb 11, 2025
…2a9b8460d Pull request: release <- staging/91f2699f-90d3-414d-8bbe-4c62a9b8460d
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation and Context
Continue updating the API to pass
Supplier<ProfileFile>to classes.Modifications
Updated services/s3 classes using
ProfileFileinternally.Testing
Added unit tests.
Screenshots (if appropriate)
Types of changes
Checklist
mvn installsucceedsscripts/new-changescript and following the instructions. Commit the new file created by the script in.changes/next-releasewith your changes.License