You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "tanmayrauth (via GitHub)" <gi...@apache.org> on 2026/05/03 16:34:56 UTC
[PR] fix(io): resolve S3 PermanentRedirect for S3 Table Buckets (#974) [iceberg-go]
tanmayrauth opened a new pull request, #975:
URL: https://github.com/apache/iceberg-go/pull/975
Two issues caused 301 PermanentRedirect errors when writing data files to S3 Table Buckets:
1. The SigV4 signing region configured on the REST catalog was not propagated to the S3 I/O layer. When s3.region is absent from vended credentials, the S3 client falls back to the SDK default region, which may differ from the bucket's actual region. Fix: toProps() now sets client.region from the SigV4 signing region as a fallback.
2. The S3 client defaulted to path-style addressing, which is deprecated by AWS. Virtual-hosted style is now the default for standard AWS S3, with path-style reserved for custom endpoints.
Fixes #974
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] fix(io): resolve S3 PermanentRedirect for S3 Table Buckets (#974) [iceberg-go]
Posted by "laskoviymishka (via GitHub)" <gi...@apache.org>.
laskoviymishka merged PR #975:
URL: https://github.com/apache/iceberg-go/pull/975
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] fix(io): resolve S3 PermanentRedirect for S3 Table Buckets (#974) [iceberg-go]
Posted by "tanmayrauth (via GitHub)" <gi...@apache.org>.
tanmayrauth commented on PR #975:
URL: https://github.com/apache/iceberg-go/pull/975#issuecomment-4372340328
Thanks for the review! Addressed all the feedback, extracted resolveUsePathStyle(endpoint, props) as a helper with table tests covering the default, override, and edge cases. Replaced the raw "client.region" string with an S3ClientRegion constant in io/config.go. Gated the SigV4 region fallback to only s3 and s3tables services, and folded it into the existing enableSigv4 block with a softened "best-effort fallback" comment. No changelog file exists in the repo so I've added a breaking-change note to the PR description instead.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org