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

forcedPathStyle was needed and removed #1098

Closed
NikolayMetchev opened this issue Nov 1, 2023 · 4 comments · Fixed by #1099
Closed

forcedPathStyle was needed and removed #1098

NikolayMetchev opened this issue Nov 1, 2023 · 4 comments · Fixed by #1099
Labels
bug This issue is a bug.

Comments

@NikolayMetchev
Copy link

Describe the bug

Hi, in the latest version of the AWS SDK forcePathStyle has been removed (f6502eb) which is a workaround mentioned #842 (comment)
This has broken the S3 client as described in that bug

Expected behavior

The s3 workaround to keep working.

Current behavior

can't use S3 client for CI use case.

Steps to Reproduce

see original bug for details

Possible Solution

put back forcedPathStyle

Context

No response

AWS Kotlin SDK version used

0.33.0-beta

Platform (JVM/JS/Native)

JVM

Operating System and version

Linux

@NikolayMetchev NikolayMetchev added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Nov 1, 2023
@NikolayMetchev
Copy link
Author

@aajtodd
Copy link
Contributor

aajtodd commented Nov 1, 2023

Thanks for the bug, indeed this appears to be accidentally removed (the intended removal should have been scoped to just shared configuration properties in ~/.aws/config).

Will work on a fix for this.

Copy link

github-actions bot commented Nov 1, 2023

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

@aajtodd aajtodd removed the needs-triage This issue or PR still needs to be triaged. label Nov 1, 2023
@aajtodd
Copy link
Contributor

aajtodd commented Nov 2, 2023

This should be fixed in 0.33.1-beta that was released yesterday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug.
Projects
None yet
2 participants