Skip to content

Read .aws/credentials#39

Merged
motdotla merged 1 commit intomotdotla:masterfrom
macoshita:patch-1
Jan 20, 2016
Merged

Read .aws/credentials#39
motdotla merged 1 commit intomotdotla:masterfrom
macoshita:patch-1

Conversation

@macoshita
Copy link
Contributor

It is OK only not set default value.

Remember, if you set your AWS credentials in the shared credentials file or via environment variables, the AWS SDK for Node.js will automatically detect them, and you will not need to perform any manual credential configuration in your application.

http://docs.aws.amazon.com/AWSJavaScriptSDK/guide/node-configuring.html

It is OK only not set default value.

> Remember, if you set your AWS credentials in the shared credentials file or via environment variables, the AWS SDK for Node.js will automatically detect them, and you will not need to perform any manual credential configuration in your application.

http://docs.aws.amazon.com/AWSJavaScriptSDK/guide/node-configuring.html
@madmod
Copy link

madmod commented Jan 11, 2016

+1 please merge!

@rafalyesware
Copy link

💯 this makes deploying much easier/safer.

@mikefrey
Copy link

@motdotla Any chance this can be merged?

@motdotla
Copy link
Owner

Thanks for this @macoshita.

motdotla added a commit that referenced this pull request Jan 20, 2016
@motdotla motdotla merged commit 05929c4 into motdotla:master Jan 20, 2016
@tlvince
Copy link

tlvince commented Apr 17, 2016

@motdotla any chance you publish a new release with this? You could also look into semantic-release to automate this. Thanks!

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.

7 participants