snazy commented on PR #10283: URL: https://github.com/apache/iceberg/pull/10283#issuecomment-2099816561
> there are cases where they are used As (not so widely) known, special characters in S3 URIs - i.e. those URIs, for example with (back)quotes and hash/question-marks, are not properly escaped by Iceberg. This makes those locations/URIs violate [RFC 2396](https://datatracker.ietf.org/doc/html/rfc2396#section-2.2) - causing trouble even with AWSSDK functionality (see for example `S3Utilities`). This change is an effort to mitigate issues with those special characters. @danielcweeks can you point to the use cases you're thinking of in the Iceberg code base? I think that not changing it will cause non-resolvable user facing issues sooner or later. -- 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-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
