adutra commented on PR #10621: URL: https://github.com/apache/iceberg/pull/10621#issuecomment-2238944292
@jackye1995 and @flyrain I also like what Jack initiated here, and I would like this to move forward. I will open another PR shortly, with both OAuth2 and SigV4 retrofitted to implement `AuthManager`. I cannot speak for Kerberos either, but given that SigV4 is a very "demanding" auth scheme, requiring full access to the outgoing request, I think we should be fine, even if Kerberos or other schemes also require access to the full request. I can however speak for OAuth2, having worked on a similar design for Nessie. I think we can build a much better OAuth support for the REST catalog using this PR as a foundation. -- 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
