adutra opened a new pull request, #11995:
URL: https://github.com/apache/iceberg/pull/11995

   5th PR for the Auth Manager API. Previous ones:
   
   * #11992
   * #11844
   * #11809
   * #11769 
   
   This PR introduces the SigV4 auth manager. It can be reviewed independently 
of [part4](https://github.com/apache/iceberg/pull/11992).
   
   The peculiarity of the SIgV4 manager is that it takes a delegate auth 
manager. IOW, it is able to first delegate authentication to some other auth 
manager (by default, OAuth2), then sign the request, potentially relocating the 
authorization header introduced by the delegate manager. **This is the current 
behavior of SigV4 signing**.
   
   **Contrary to what I said earlier on the dev list**, the SigV4 manager at 
this stage is still "unplugged". Merging this PR is completely harmless.
   
   The old `RESTSigV4Signer` is still there and active, its contents can be 
compared with those of the new `RESTSigV4AuthSession` that will replace it soon.
   
   \cc @nastra @danielcweeks 


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

Reply via email to