sungwy commented on issue #2862: URL: https://github.com/apache/iceberg-python/issues/2862#issuecomment-3747463555
I ran into this while reviewing the AuthManager refactor work on the Java/Catalog side. In the [OAuth2AuthManager V2 design doc](https://docs.google.com/document/d/1Hxw-t8Maa7wZFmrlSujm7LRawKsFP3Q31tET_3aRnQU/edit?tab=t.0#heading=h.hjhenfg8d08l), @adutra calls out the same issue. The S3 signer is tightly coupled to legacy OAuth2 APIs, even though it really needs to live closer to the REST catalog and reuse its auth state. As AuthManager evolves, it would be good to align on a common pattern where the signer is created at the catalog level and passed into FileIO, rather than being constructed from FileIO properties. I think it would be good to have a consistent, pluggable signer model across languages. -- 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]
