babashka / pod-babashka-aws

Deprecated, use https://github.com/grzm/awyeah-api
Apache License 2.0
61 stars 14 forks source link

awsSession is not passed into system-properties #21

Closed digash closed 3 years ago

digash commented 3 years ago

14 missed "aws.sessionToken" system property that is needed for auth to work

It was not working upstream either, they fixed it cognitect-labs/aws-api#167 Released in aws-api 0.8.498

I can submit a pull request but it is trivial change, just need to add it to the list at credentials.clj line 187

Also would be good to update deps.edn aws-api reference to 0.8.498 version.

borkdude commented 3 years ago

@digash PR welcome. I'll update the deps tomorrow and will make a new release with your PR included.

digash commented 3 years ago

@borkdude pull request created

borkdude commented 3 years ago

Thanks. You could try if this works for you with a build from CI, if you want to test pre-release.