sungwy opened a new pull request, #18110: URL: https://github.com/apache/iceberg/pull/18110
Draft, not to be merged before https://github.com/apache/iceberg/pull/18080 Reading signed URLs a server returns in a plan response is https://github.com/apache/iceberg/pull/17457 - this PR does not depend on it. This PR adds: - `SupportsPreSigning` which returns one signed URL per location, signed for GET. - `RemoteSigningClient` which is a client of `signRequest`, and is built from table FileIO's properties. - `S3FileIO` and `GCSFileIO` updated to implement `SupportsPreSigning` Notes for Reviewer: - The FileIO owns the request because region is a runtime property of its storage client - expiry of the signature is the catalog's implementation detail and must be communicated out of band, or inferred from the url parameters. - `S3V4RestSignerClient` could be converged into `RemoteSigningClient` as a followup -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
