Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EC2, Enable authentication using default credential chain #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

EC2, Enable authentication using default credential chain #3

wants to merge 2 commits into from

Conversation

tluimes
Copy link

@tluimes tluimes commented Jan 21, 2021

Signed-off-by: Tim Luimes [email protected]

Enables the ability for the ec2 client session to be authenticated using methods provided by the aws SDK's default credential chain when the accessKey and secretKey aren't provided.

https://docs.aws.amazon.com/sdk-for-go/v1/developer-guide/configuring-sdk.html
"When you initialize a new service client without providing any credential arguments, the SDK uses the default credential provider chain to find AWS credentials. The SDK uses the first provider in the chain that returns credentials without an error. The default provider chain looks for credentials in the following order:

Environment variables.

Shared credentials file.

If your application uses an ECS task definition or RunTask API operation, IAM role for tasks.

If your application is running on an Amazon EC2 instance, IAM role for Amazon EC2."

@derek
Copy link

derek bot commented Jan 21, 2021

Thank you for your contribution. I've just checked and your commit doesn't appear to be signed-off. That's something we need before your Pull Request can be merged. Please see our contributing guide.
Tip: if you only have one commit so far then run: git commit --amend --signoff and then git push --force.

@derek derek bot added the no-dco label Jan 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant