Use ipaddress module for no_proxy parsing - #1033

I cannot add my changes to this to fix failing CI @jplitza because I don't have proper permissions. But you need to apply black formatting and fix the AssertionError in test_connect to make CI pass.

If this is not changed in 2 weeks, I will make a duplicate PR with a similar edit, with the CI fixes.

That should do it. Those black findings were both outside what I changed, so I hoped rebasing would fix it, but it didn't.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.97%. Comparing base (981c00e) to head (cc5a372).
⚠️ Report is 31 commits behind head on master.

@@            Coverage Diff             @@
##           master    #1033      +/-   ##
==========================================
+ Coverage   82.59%   84.97%   +2.37%     
==========================================
  Files          13       14       +1     
  Lines        1396     1484      +88     
  Branches      264      279      +15     
==========================================
+ Hits         1153     1261     +108     
+ Misses        163      121      -42     
- Partials       80      102      +22     

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

Thanks for the PR, and the quick fix!