[PR #8445 backport][3.9] Fixes read_timeout on WS connection not respecting ws_connect's timeouts #8444

[arcivanov]

Please see #8445 for the source PR

[bdraco]

The bot will auto-create backport PRs (assuming there are no conflicts) from #8445 once it merges since its tagged with the backport tags.

[arcivanov]

@bdraco there will be major conflicts as the API changed (ClientWSTimeout introduced in master vs 3.9)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.42%. Comparing base (e4a63ff) to head (db19243).
Report is 2 commits behind head on 3.9.

Additional details and impacted files

@@            Coverage Diff             @@
##              3.9    #8446      +/-   ##
==========================================
- Coverage   97.43%   97.42%   -0.02%     
==========================================
  Files         108      108              
  Lines       32884    32955      +71     
  Branches     3908     3915       +7     
==========================================
+ Hits        32042    32105      +63     
- Misses        633      639       +6     
- Partials      209      211       +2     

Flag	Coverage Δ
CI-GHA	97.33% <100.00%> (-0.02%)	⬇️
OS-Linux	97.03% <100.00%> (+<0.01%)	⬆️
OS-Windows	94.59% <100.00%> (+0.01%)	⬆️
OS-macOS	?
Py-3.10.11	94.41% <100.00%> (+0.01%)	⬆️
Py-3.10.14	96.78% <100.00%> (-0.03%)	⬇️
Py-3.11.9	97.04% <100.00%> (-0.02%)	⬇️
Py-3.12.3	97.09% <100.00%> (-0.02%)	⬇️
Py-3.8.10	94.38% <100.00%> (+0.01%)	⬆️
Py-3.8.18	96.75% <100.00%> (+<0.01%)	⬆️
Py-3.9.13	94.39% <100.00%> (+<0.01%)	⬆️
Py-3.9.19	96.77% <100.00%> (-0.03%)	⬇️
Py-pypy7.3.15	?
Py-pypy7.3.16	96.33% <100.00%> (?)
VM-macos	?
VM-ubuntu	97.03% <100.00%> (+<0.01%)	⬆️
VM-windows	94.59% <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.

[Dreamsorcerer]

@bdraco there will be major conflicts as the API changed (ClientWSTimeout introduced in master vs 3.9)

Yes, but it's preferable to follow our normal backport procedure to ensure everything lines up with cherry picking etc. (for example, this needs to backport to 3.10 first).

[bdraco]

3.9 is no longer getting backports. I'm going to close this one since we are going to do a 3.10 release soon