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

[C++] Support AWS S3 Web identity credentials #26628

Closed
asfimport opened this issue Nov 21, 2020 · 7 comments
Closed

[C++] Support AWS S3 Web identity credentials #26628

asfimport opened this issue Nov 21, 2020 · 7 comments

Comments

@asfimport
Copy link
Collaborator

asfimport commented Nov 21, 2020

It seems to me that Arrow only supports at the moment the "AssumeRole" AWS STS API, but not the other options offered:

Reporter: Paul Balanca / @balancap
Assignee: Sahil Gupta / @sahil1105

Related issues:

PRs and other links:

Note: This issue was originally created as ARROW-10675. Please see the migration documentation for further details.

@asfimport
Copy link
Collaborator Author

Antoine Pitrou / @pitrou:
Thank you for the report. Would you like to provide a pull request, at least on the C++ side?

@asfimport
Copy link
Collaborator Author

Paul Balanca / @balancap:
I should be able to draft something next week. I may need a bit of guidance on the testing strategy you're adopting in the project of this kind of authentication code.

@asfimport
Copy link
Collaborator Author

Antoine Pitrou / @pitrou:
Feel free to ask any questions. For testing, though, my intuition is that we can't do much for integration testing of such a feature, at least not with our current CI.

@asfimport
Copy link
Collaborator Author

Antoine Pitrou / @pitrou:
Issue resolved by pull request 10088
#10088

@asfimport
Copy link
Collaborator Author

Tobias Ullrich:
Hi @pitrou

we just tried to use this (as we need it in a k8s setup) and it does not work in pyarrow. Looking at the code and both PRs, it seems  like PR #10088 is actually not adding the capabilities in Python. PR #8977 seems to would have added this.

Please let me know if I am missing something here.

Thank you.

-Tobias

@asfimport
Copy link
Collaborator Author

Antoine Pitrou / @pitrou:
[~tobias_ullrich] Sorry, you are right. Python support was deferred to a later JIRA/PR and this was forgotten when closing this JIRA. Will open another one.

@asfimport
Copy link
Collaborator Author

Antoine Pitrou / @pitrou:
I opened ARROW-16457 for the Python side.

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

No branches or pull requests

1 participant