Skip to content
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

Depend on com.amazonaws.aws-java-sdk-sts #86

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zyga
Copy link

@zyga zyga commented Feb 3, 2023

This allows using real AWS S3 with a web identity token. See [1] for details. Absence of this dependency requires using less desired password-based authentication.

[1] https://github.com/aws/aws-sdk-java/blob/master/aws-java-sdk-core/src/main/java/com/amazonaws/auth/profile/internal/securitytoken/STSProfileCredentialsServiceLoader.java#L20

Signed-off-by: Zygmunt Krynicki [email protected]

This allows using real AWS S3 with a web identity token. See [1] for details.
Absence of this dependency requires using less desired password-based
authentication.

[1] https://github.com/aws/aws-sdk-java/blob/master/aws-java-sdk-core/src/main/java/com/amazonaws/auth/profile/internal/securitytoken/STSProfileCredentialsServiceLoader.java#L20

Signed-off-by: Zygmunt Krynicki <[email protected]>
@hawkbit-bot
Copy link

Can one of the admins verify this patch?

@forsberg
Copy link

Was trying to use Web Based authentication with a custom AWS IAM Role when running hawkbit in AWS EKS Kubernetes cluster. Doing this is the best practice when it comes to providing AWS permissions for pods, but without this fix, it's not possible.

I.e, I confirm this is an important patch.

@zyga
Copy link
Author

zyga commented Jun 22, 2023

@stefbehl could we merge this?

@RomanGrekov
Copy link

Still actual

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.

4 participants