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

feat: improve support for arbitrary s3 endpoints and for multiple aws profiles #240

Merged
merged 1 commit into from
Aug 1, 2022

Conversation

starpit
Copy link
Contributor

@starpit starpit commented Aug 1, 2022

  • we had been assuming that the ~/.aws/credentials file had only one credential pair
  • and had been hard-coding an s3 endpoint

on the latter, there isn't a supported solution for this yet (aws/aws-cli#1270). lacking a supported solution, this PR for now assumes that the ~/.aws/config section may have an endpoint_url= conf entry.

… profiles

- we had been assuming that the ~/.aws/credentials file had only one credential pair
- and had been hard-coding an s3 endpoint

on the latter, there isn't a supported solution for this yet (aws/aws-cli#1270). lacking a supported solution, this PR for now assumes that the ~/.aws/config section may have an `endpoint_url=` conf entry.
@starpit starpit merged commit 14ad134 into guidebooks:main Aug 1, 2022
@starpit starpit deleted the s3-profs branch August 1, 2022 15:52
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.

1 participant