-
Notifications
You must be signed in to change notification settings - Fork 9
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
Document how to add s3 credentials #6
Comments
For those, who are interested in this as well and have found only this issue |
Most of the time you're going to want to use the AWS SDK ProviderChain to grab these from environment variables. Going through the catalog is somewhat of an escape hatch for when you can't use the env - it's less ideal because you're pushing credentials into ZooKeeper. |
I generally don't like the idea to access environment variables here and there. I prefer to capture all of them in my config and to pass them explicitly further. |
But your point about ZooKeeper is valid and important if you are exposing your ZooKeeper out of your private network. |
No description provided.
The text was updated successfully, but these errors were encountered: