Rename from connection_timeout -> connect_timeout

[staticfloat]

I think this may be a typo; I'm a little surprised there's not a warning or something that you're using a useless keyword argument.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7bdb45c) 82.69% compared to head (37be604) 82.69%.
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1131   +/-   ##
=======================================
  Coverage   82.69%   82.69%           
=======================================
  Files          32       32           
  Lines        3052     3052           
=======================================
  Hits         2524     2524           
  Misses        528      528           

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

[fredrikekre]

kwargs... are just forwarded down the chain at all stages I think. Perhaps at the end there should be a check that all kwargs have been picked up by some layer below.

Edit: #1024