Skip to content

#7 Add support for session expiration - #8

Open
tomisaacson wants to merge 1 commit into
whereisaaron:masterfrom
parsley72:master
Open

#7 Add support for session expiration#8
tomisaacson wants to merge 1 commit into
whereisaaron:masterfrom
parsley72:master

Conversation

@tomisaacson

Copy link
Copy Markdown

No description provided.

@virgilwashere virgilwashere left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  • Update README.md?

I've also included support for AWS_SESSION_EXPIRATION my branch, but it's on top of PR #10 and a pending PR

Virgil

Comment thread get-aws-profile.sh
fi

if [[ "${SHOW_KEY}" = false && "${SHOW_SECRET}" = false && "${SHOW_SESSION_TOKEN}" = false ]]; then
if [[ "${SHOW_KEY}" = false && "${SHOW_SECRET}" = false && "${SHOW_SESSION_TOKEN}" = false && "${SHOW_EXPIRATION}" = false ]]; then

@virgilwashere virgilwashere Jul 14, 2019

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

$SHOW_EXPIRATION is probably meant to be $SHOW_SESSION_EXPIRATION

@whereisaaron

Copy link
Copy Markdown
Owner

@tomisaacson I've been convinced to add this as part of #10. Sorry to pass over your PR, which was first. Would you mind reviewing #10 instead?

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