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

Add initial support for IAM roles. #26

Merged
merged 1 commit into from Dec 4, 2013

Conversation

Projects
None yet
2 participants
@jimbrowne

jimbrowne commented Apr 1, 2013

An attempt to fix https://rt.cpan.org/Public/Bug/Display.html?id=81664
Note that temporary credentials should be checked for expiration and
re-fetched from the metadata service.

@jimbrowne

This comment has been minimized.

Show comment
Hide comment
@jimbrowne

jimbrowne Apr 1, 2013

I do need to add one of:

  • Checking for credential expiration based on the timestamp and re-fetch the credentials
  • Watch for 403 permission errors and re-try the request after re-fetching credentials. (This is what boto does.)

I opened this request now, though, to get discussion rolling.

jimbrowne commented Apr 1, 2013

I do need to add one of:

  • Checking for credential expiration based on the timestamp and re-fetch the credentials
  • Watch for 403 permission errors and re-try the request after re-fetching credentials. (This is what boto does.)

I opened this request now, though, to get discussion rolling.

@mrallen1

This comment has been minimized.

Show comment
Hide comment
@mrallen1

mrallen1 Nov 15, 2013

Owner

I'm sorry I let this linger - would you rebase this patch please? I'd like to get it in to the module.

Owner

mrallen1 commented Nov 15, 2013

I'm sorry I let this linger - would you rebase this patch please? I'd like to get it in to the module.

Add initial support for IAM roles.
An attempt to fix https://rt.cpan.org/Public/Bug/Display.html?id=81664
Note that temporary credentials should be checked for expiration and
re-fetched from the metadata service.
@jimbrowne

This comment has been minimized.

Show comment
Hide comment
@jimbrowne

jimbrowne Dec 4, 2013

Rebased against current master

jimbrowne commented Dec 4, 2013

Rebased against current master

mrallen1 added a commit that referenced this pull request Dec 4, 2013

Merge pull request #26 from jimbrowne/temp-security-creds
Add initial support for IAM roles.

@mrallen1 mrallen1 merged commit 8b8759a into mrallen1:master Dec 4, 2013

@mrallen1

This comment has been minimized.

Show comment
Hide comment
@mrallen1

mrallen1 Dec 4, 2013

Owner

Thank you very much

Owner

mrallen1 commented Dec 4, 2013

Thank you very much

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