danielcweeks commented on PR #8400: URL: https://github.com/apache/iceberg/pull/8400#issuecomment-1694742987
> Our understanding is that the existing Authentication flow in PyIceberg is actually an [ROPC](https://stackoverflow.com/questions/22077487/difference-between-the-resource-owner-password-flow-and-the-client-credential) I don't necessarily agree with this interpretation as the original spec was written with the expectation that the REST Catalog is the Auth Server. I see that this could be interpreted differently if you already have an external auth server and are attempting to use that, but that is was not the intent. We've discussed and are open to allowing for setting the url independently, but I think we need to update the REST spec to reflect that before we change the clients so as not to introduce inconsistent or incompatible changes/behaviors outside of what is defined in the spec. -- 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]
