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

Fix issue requesting current EC2 role #1738

Closed
wants to merge 1 commit into from
Closed

Fix issue requesting current EC2 role #1738

wants to merge 1 commit into from

Conversation

dspangen
Copy link
Contributor

EC2 appears to have changed how you access the collection members of instance metadata--you now need a / after each collection to list them:

curl http://169.254.169.254/latest/meta-data/iam/security-credentials # nothing
curl http://169.254.169.254/latest/meta-data/iam/security-credentials/
ec2-staging-es

EC2 appears to have changed how you access the collection members of instance metadata--you know need a `/` after each collection to list them:
```
curl http://169.254.169.254/latest/meta-data/iam/security-credentials # nothing
curl http://169.254.169.254/latest/meta-data/iam/security-credentials/
ec2-staging-es
```
@remh
Copy link
Contributor

remh commented Jun 29, 2015

Thanks a lot @dspangen do you know how it will play with older EC2 instances ?
Is that a change that got deployed only on newly created instances or to all instances ?

@dspangen
Copy link
Contributor Author

It started breaking on instances we've had deployed for over a year.

@olivielpeau
Copy link
Member

Thanks @dspangen for contributing! I've opened a new PR that rebases your commit and adds a trailing slash to the public-keys endpoint (#1750), so I'm going to close this one.

@olivielpeau olivielpeau closed this Jul 6, 2015
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.

3 participants