DEFAULT_CONNECT_TIMEOUT = 10.0

[abhinavsingh]

Having a separate DEFAULT_CONNECT_TIMEOUT allows us to configure different values for connect and data timeout values. Currently, in this PR, this is hardcoded to 10.0 and cannot be overridden using flags currently.

Without this, overriding DEFAULT_TIMEOUT would automatically also apply during the connect phase leading into longer connect timeouts

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.67%. Comparing base (d502349) to head (32d7d5c).
Report is 1 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #1519   +/-   ##
========================================
  Coverage    84.66%   84.67%           
========================================
  Files          186      186           
  Lines         8420     8424    +4     
  Branches      1279     1279           
========================================
+ Hits          7129     7133    +4     
  Misses        1086     1086           
  Partials       205      205           

Flag	Coverage Δ
GHA	84.52% <100.00%> (+<0.01%)	⬆️
Linux	84.16% <100.00%> (+<0.01%)	⬆️
Python 3.10	85.19% <100.00%> (+<0.01%)	⬆️
Python 3.11	84.41% <100.00%> (+<0.01%)	⬆️
Python 3.12	84.41% <100.00%> (+<0.01%)	⬆️
Python 3.6	85.15% <100.00%> (+<0.01%)	⬆️
Python 3.7	85.15% <100.00%> (+<0.01%)	⬆️
Python 3.8	85.18% <100.00%> (+<0.01%)	⬆️
Python 3.9	85.25% <100.00%> (+<0.01%)	⬆️
Windows	77.97% <100.00%> (+0.03%)	⬆️
macOS	84.48% <100.00%> (+<0.01%)	⬆️
pytest	84.52% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.