Skip to content

Conversation

@eersinyildizz
Copy link
Contributor

Motivation and Context (Why the change? What's the scenario?)

This PR adds support for ForcePathStyle configuration in AWSS3Config

High level description (Approach, Design)

  • Added a ForcePathStyle boolean property to AWSS3Config (default: false)
  • Updated AWSS3Storage to pass the value to the AmazonS3Config.ForcePathStyle property
  • Maintains backward compatibility with AWS S3 by leaving the default behavior unchanged
  • Improves compatibility with services like MinIO, LocalStack, and others that require path-style requests

This change is fully backward compatible and has no effect on current configurations unless explicitly enabled.

@eersinyildizz eersinyildizz requested a review from dluc as a code owner March 24, 2025 12:49
@eersinyildizz
Copy link
Contributor Author

@eersinyildizz please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@eersinyildizz
Copy link
Contributor Author

@microsoft-github-policy-service agree

@dluc dluc merged commit 1c424ed into microsoft:main Mar 24, 2025
11 checks passed
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.

2 participants