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

Support passing AWS session token via environment variable #226

Merged
merged 2 commits into from
Oct 26, 2022
Merged

Support passing AWS session token via environment variable #226

merged 2 commits into from
Oct 26, 2022

Conversation

matshch
Copy link
Contributor

@matshch matshch commented Oct 24, 2022

desync already supports taking aws_session_token from shared credentials file, but it is not convenient to store token there if it is received by another application that will call desync. This commit allows to pass session token via environment variable like other credentials.

Also while we there, lets add support for the same environment variables as AWS CLI and AWS SDKs accept.

Co-authored-by: Aleksejs Lucko lu4ik@yandex-team.ru

desync already supports taking aws_session_token from shared credentials
file, but it is not convenient to store token there if it is received by
another application that will call desync. This commit allows to pass
session token via environment variable like other credentials.

Co-authored-by: Aleksejs Lucko <lu4ik@yandex-team.ru>
@shcheglovnd
Copy link

Good job!

README.md Outdated Show resolved Hide resolved
Most applications and libraries support same environment variables as
AWS CLI and AWS SDKs. It will be more convenient if desync will also
support them.

Co-authored-by: Aleksejs Lucko <lu4ik@yandex-team.ru>
Copy link
Owner

@folbricht folbricht left a comment

Choose a reason for hiding this comment

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

Great, thank you

@folbricht folbricht merged commit a883b9b into folbricht:master Oct 26, 2022
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.

3 participants