[GitHub] [iceberg] dhruv-pratap commented on pull request #6019: Python: Add support for providing SSL config for REST Catalog client.

[GitBox <gi...@apache.org>]

dhruv-pratap commented on PR #6019:
URL: https://github.com/apache/iceberg/pull/6019#issuecomment-1284332682

   High level summary of changes done as part of the PR:
   1. Rest Catalog encapsulates a `requests.Session` 
   2. Configure `Session` object with SSL config, if provided, during initialization.
   3. Configure existing headers one-time in the `session` object during initialization.
   4. Use the one-time initialized `Session` object for all further REST calls.
   5. Add unit tests for SSL config initiation, and enforce HTTP header checks in existing REST mock tests.


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org