search

ex-aws / ex_aws_sts

13 stars 31 forks source link

Change perform action to request action #4

Closed Theuns-Botha closed 5 years ago

benwilson512 commented 5 years ago

Can you elaborate on why you're making this change?

Theuns-Botha commented 5 years ago

Good day, sorry for the late reply, I was unfortunately busy with all kinds of stuff. What happened was that the role was effectively assumed(i.e. the temporary credentials were fetched), but they were never actually updated when they expired, although there was code in place to do the update. This change stopped the "refresh" logic from being bypassed.

Theuns-Botha commented 5 years ago

I suppose it is too late to want to merge this now though.