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

s3: remove hardcoded host #673

Merged
merged 1 commit into from
Apr 26, 2023

Conversation

honzasterba
Copy link
Contributor

fixes #672

@geemus
Copy link
Member

geemus commented Apr 21, 2023

Thanks. I'm struggling to remember why that change was made in the first place, looks like it was back in 2019. My only concern is if this needs to always go to the main s3 host instead of a region-based host. We might need to be more sophisticated about it (ie any of the aws stuff should maybe map to this value, instead of the regional one). Are you able to test to see what happens here?

@honzasterba
Copy link
Contributor Author

updated to use the default region host unless @host has been set

@geemus
Copy link
Member

geemus commented Apr 26, 2023

Thanks, I think/hope that should cover our bases.

@geemus geemus merged commit b8e9a07 into fog:master Apr 26, 2023
@geemus
Copy link
Member

geemus commented May 18, 2023

Released in v3.19.0.

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.

bug: directories method ignores :endpoint config
2 participants