[ ] I am adding a new automated test(s) to verify correctness of my new code
[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am modifying authorization mechanisms
[ ] I am adding new credentials
[ ] I am modifying OCSP code
[ ] I am adding a new dependency
Please describe how your code solves the related issue.
By default, request do not time out unless a timeout value is set explicitly. A user recently reported that their file transfer would sometimes hang indefinitely at reading from a request call. This PR aims to mitigate this issue by specifying a default connection timeout of 10 seconds and socket read timeout of 10 minutes. Please see
https://requests.readthedocs.io/en/latest/user/advanced/#timeouts
Please answer these questions before submitting your pull requests. Thanks!
What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes SNOW-1468259
Fill out the following pre-review checklist:
Please describe how your code solves the related issue.
By default,
request
do not time out unless a timeout value is set explicitly. A user recently reported that their file transfer would sometimes hang indefinitely at reading from a request call. This PR aims to mitigate this issue by specifying a default connection timeout of 10 seconds and socket read timeout of 10 minutes. Please see https://requests.readthedocs.io/en/latest/user/advanced/#timeouts