Skip to content
This repository was archived by the owner on Apr 30, 2019. It is now read-only.

Use latest AWS SDK and allow profile credentials#42

Open
heroldus wants to merge 3 commits into
spring-attic:masterfrom
Scout24:master
Open

Use latest AWS SDK and allow profile credentials#42
heroldus wants to merge 3 commits into
spring-attic:masterfrom
Scout24:master

Conversation

@heroldus

Copy link
Copy Markdown

I just updated AWS SDK dependency and allow profile credentials for authentication.

@mjallday

Copy link
Copy Markdown

this looks like it supersedes #38.

👍 this looks just like what i want. this fixes #28 too.

@spring-buildmaster @nebhale what can we do to help get this merged for the next release?

this shouldn't be a breaking change. the provider chain runs through the providers in order and since the EnvironmentVariableCredentialsProvider comes first if those are set as well as the AWS_PROFILE environment variable it should not change the behavior.

@mjallday

Copy link
Copy Markdown

@spring-buildmaster @spring-issuemaster @nebhale is this project still maintained? this would be a very handy fix to merge in.

@huima

huima commented Mar 11, 2016

Copy link
Copy Markdown

This is done here: https://github.com/Yleisradio/aws-maven

@wrasdf

wrasdf commented Jul 21, 2016

Copy link
Copy Markdown

bump

@mattjtodd

Copy link
Copy Markdown

@nebhale is there any possibility of adding some new maintainers for this project? I'd be happy to volunteer to help out.

@ddcprg

ddcprg commented Jan 12, 2018

Copy link
Copy Markdown

new maintainers would be great! we need an upgrade similar to what's on this branch

@mjallday

Copy link
Copy Markdown

@ddcprg if you’re interested we’re maintaining a fork over at https://github.com/verygoodsecurity/aws-maven/ which has a later version of the aws cli and supports this change already.

@ddcprg

ddcprg commented Jan 15, 2018

Copy link
Copy Markdown

Thanks @mjallday I'll take a look at your fork

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

6 participants