raphaelsolarski commented on code in PR #17148:
URL: https://github.com/apache/iceberg/pull/17148#discussion_r3573844772


##########
core/src/main/java/org/apache/iceberg/rest/auth/OAuth2Util.java:
##########


Review Comment:
   For now it is intentional - in fromCredential it is obvious that this header 
should not be passed. But in case of token exchange I'm not sure because there 
are multiple scenarios how token exchange request can be authenticated. I need 
a bit of time to reason about it.



-- 
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]

Reply via email to