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

feat(aws): add a fallback for expiration key #4455

Merged

Conversation

liangyungong
Copy link
Contributor

@liangyungong liangyungong commented Oct 8, 2022

Description

For third party login solution like saml2aws. It uses x_security_token_expires as the expiration key name.

Motivation and Context

Add x_security_token_expires as a fallback, so both expiration and x_security_token_expires works.

Closes #4414

Screenshots (if appropriate):

How Has This Been Tested?

  • I have tested using MacOS
  • I have tested using Linux
  • I have tested using Windows

Checklist:

  • I have updated the documentation accordingly.
  • I have updated the tests accordingly.

Copy link
Member

@davidkna davidkna left a comment

Choose a reason for hiding this comment

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

I have made two suggestions regarding the tests. Please also update the docs in docs/config/README.md.

src/modules/aws.rs Outdated Show resolved Hide resolved
src/modules/aws.rs Outdated Show resolved Hide resolved
- extend the time range from `-2s,0s` to `-5s,+2s`
docs/config/README.md Outdated Show resolved Hide resolved
Co-authored-by: David Knaack <davidkna@users.noreply.github.com>
@liangyungong
Copy link
Contributor Author

@davidkna Would someone merge this soon? 🤔

@davidkna davidkna merged commit 5a2c85d into starship:master Oct 15, 2022
@liangyungong liangyungong deleted the liangyun/aws-expiration-keyword-fallback branch October 16, 2022 02:33
Indyandie pushed a commit to Indyandie/starship that referenced this pull request Jul 26, 2023
* feat(aws): add a fallback for `expiration`

* fix(aws): intermittent test failures

- extend the time range from `-2s,0s` to `-5s,+2s`

* fix: `docs/config/README.md` readability

Co-authored-by: David Knaack <davidkna@users.noreply.github.com>

Co-authored-by: David Knaack <davidkna@users.noreply.github.com>
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.

None yet

2 participants