refactor: migrate from urllib to requests

[BreadGenie]

• fix Refactor: requests library vs urlib* #1311

[codecov-commenter]

Codecov Report

Merging #1569 (2285ef1) into main (7dd0d75) will increase coverage by 2.77%.
The diff coverage is 65.21%.

@@            Coverage Diff             @@
##             main    #1569      +/-   ##
==========================================
+ Coverage   80.34%   83.11%   +2.77%     
==========================================
  Files         281      281              
  Lines        5581     5574       -7     
  Branches      911      913       +2     
==========================================
+ Hits         4484     4633     +149     
+ Misses        899      750     -149     
+ Partials      198      191       -7     

Flag	Coverage Δ
longtests	83.11% <65.21%> (+2.77%)	⬆️

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

Impacted Files	Coverage Δ
cve_bin_tool/checkers/sqlite.py	57.44% <20.00%> (+0.30%)	⬆️
cve_bin_tool/version.py	64.70% <50.00%> (-3.72%)	⬇️
cve_bin_tool/available_fix/redhat_cve_tracker.py	80.35% <75.00%> (+9.66%)	⬆️
cve_bin_tool/available_fix/debian_cve_tracker.py	83.67% <100.00%> (+5.67%)	⬆️
test/test_json.py	90.00% <100.00%> (+20.00%)	⬆️
test/utils.py	93.54% <100.00%> (ø)
cve_bin_tool/nvd_api.py	81.00% <0.00%> (-5.00%)	⬇️
cve_bin_tool/helper_script.py	41.29% <0.00%> (ø)
cve_bin_tool/version_scanner.py	57.42% <0.00%> (+0.99%)	⬆️
cve_bin_tool/extractor.py	65.21% <0.00%> (+1.24%)	⬆️
... and 11 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7dd0d75...2285ef1. Read the comment docs.

[terriko]

This is looking great. I think you're right about .ConnectionError not being the right one, but if we address that this should be ready to merge.

[terriko]

Looks good to me. Thanks for doing this! I see CI got rate limited in the long tests but otherwise looks fine.