Skip to content

storage: use session path style in HeadBucket call - #411

Merged
seruman merged 5 commits into
masterfrom
getbucketregion-virtualhost
Mar 25, 2022
Merged

seruman merged 5 commits into
masterfrom
getbucketregion-virtualhost

Conversation

@seruman

@seruman seruman commented Feb 22, 2022

Copy link
Copy Markdown
Contributor

Resolves #314

@seruman
seruman requested review from a team, aykutfarsak and igungor and removed request for a team February 22, 2022 05:31
Comment thread storage/s3.go
Why Request's S3ForcePathStyle and Credentials configs are overwritten
with the current session config's values.
@igungor

igungor commented Mar 14, 2022

Copy link
Copy Markdown
Member

@seruman could you also update the changelog please?

@seruman

seruman commented Mar 14, 2022

Copy link
Copy Markdown
Contributor Author

@seruman could you also update the changelog please?

Updated.

Comment thread CHANGELOG.md Outdated
Co-authored-by: H. İbrahim Güngör <[email protected]>
@seruman
seruman merged commit 60b1ac1 into master Mar 25, 2022
@seruman
seruman deleted the getbucketregion-virtualhost branch March 25, 2022 14:15
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.

s3 virtual host access resolution inconsistent

3 participants