search

victorskl / yawsso

Yet Another AWS SSO - sync up AWS CLI v2 SSO login session to legacy CLI v1 credentials
MIT License
302 stars 26 forks source link

Add aws_security_token to credentials #46

Closed itsamenathan closed 3 years ago

itsamenathan commented 3 years ago

Boto2 doesn't recognize aws_session_token. This PR adds aws_security_token to the credentials file which makes boto2 happy.

Link to the boto2 issue. https://github.com/boto/boto/issues/3298

codecov[bot] commented 3 years ago

Codecov Report

Merging #46 (6bba7b9) into main (4d80246) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##              main       #46   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines          368       369    +1     
=========================================
+ Hits           368       369    +1
Flag Coverage Δ
unittests 100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
yawsso/cli.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 4d80246...6bba7b9. Read the comment docs.

victorskl commented 3 years ago

Thanks for contrib. Sorry for delay. It will come with 0.6.2 release in a tick.

victorskl commented 3 years ago

GA now. Please upgrade to pip install -U yawsso==0.6.2. Thanks