010Soham commented on PR #2846: URL: https://github.com/apache/iceberg-python/pull/2846#issuecomment-3713511706
Thanks everyone for your suggestions and testing things out @Fokko I went through all the pending comments and addressed them: - S3V4RestSigner now prefers auth_manager.auth_header() (called directly) and falls back to token only if no auth manager is present. - Moved AUTH_MANAGER to auth.py and updated imports. All CI checks are green now -- 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]
