Skip to content

Add support for aws login authentication.#5776

Merged
teo-tsirpanis merged 2 commits intomainfrom
teo/core-518-add-support-for-aws-login-authentication
Mar 16, 2026
Merged

Add support for aws login authentication.#5776
teo-tsirpanis merged 2 commits intomainfrom
teo/core-518-add-support-for-aws-login-authentication

Conversation

@teo-tsirpanis
Copy link
Copy Markdown
Member

This PR adds an instance of the newly added Aws::Auth::LoginCredentialsProvider class to our custom default AWS provider chain. This enables the S3 VFS to obtain credentials configured with the aws cli command.

Validated locally by running an example after modifying it to write to S3.


TYPE: FEATURE
DESC: Added support for authenticating to S3 using console credentials, configured with the aws login CLI command.

// going to be added upstream soon with
// https://github.com/aws/aws-sdk-cpp/pull/2860, let's not update it for now.
AddProvider(make_shared<SSOCredentialsProvider>(HERE()));
if (clientConfig) {
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This condition is always true in practice.

@teo-tsirpanis teo-tsirpanis requested a review from ypatia March 12, 2026 14:53
Copy link
Copy Markdown
Member

@ypatia ypatia left a comment

Choose a reason for hiding this comment

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

LGTM! Let's also update docs in AWSCredentialsProviderChain.h.

@teo-tsirpanis teo-tsirpanis merged commit 8110ddd into main Mar 16, 2026
56 checks passed
@teo-tsirpanis teo-tsirpanis deleted the teo/core-518-add-support-for-aws-login-authentication branch March 16, 2026 17:24
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