8000 Add role option for S3 URLs by 02strich · Pull Request #275 · hashicorp/go-getter · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add role option for S3 URLs #275

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

Closed
wants to merge 1 commit into from
Closed

Conversation

02strich
Copy link

Add support for specifying an IAM role that is assumed before making S3
requests. Beforehand this was only possible when using profiles which
required local files for configuration.

Now it is possible to specify such an IAM role in the URL, thereby
supporting settings where a IAM role needs to be assumed for data
access (e.g. cross-account).

Example of usage:
s3::bucket.s3.amazonaws.com/file?aws_iam_role=some-role-arn

This currently only has a negative test case as I do not have access to the used AWS account. If someone could help me with details to write a positive test case as well, that would be much appreciated.

Add support for specifying an IAM role that is assumed before making S3
requests. Beforehand this was only possible when using profiles which
required local files for configuration.

Now it is possible to specify such an IAM role in the URL, thereby
supporting settings where a IAM role needs to be assumed for data
access (e.g. cross-account).

Example of usage:
    s3::bucket.s3.amazonaws.com/file?aws_iam_role=some-role-arn
@02strich
Copy link
Author

Any chance to get some feedback on this?

< 8000 include-fragment loading="lazy" src="/hashicorp/go-getter/issue_comments/711445366/edit_form?textarea_id=issuecomment-711445366-body&comment_context=" data-nonce="v2:e39132dd-89e3-f6df-7ed9-143b4608c80c" data-view-component="true" class="previewable-comment-form js-comment-edit-form-deferred-include-fragment">

@mdeggies mdeggies changed the base branch from master to main October 23, 2020 00:53
@02strich
Copy link
Author

Any chance to get someone to look at this?

@02strich
Copy link
Author
02strich commented Jan 1, 2021

ping?

@hashicorp-cla
Copy link
hashicorp-cla commented Mar 12, 2022

CLA assistant check
All committers have signed the CLA.

@02strich 02strich closed this Aug 29, 2023
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.

2 participants
0