feat(plugins): add support for httpx in B113
#1060
Conversation
mkniewallner
requested review from
ericwb,
lukehinds and
sigmavirus24
as code owners
| requests.get('https://gmail.com') | ||
| requests.get('https://gmail.com', timeout=None) | ||
| requests.get('https://gmail.com', timeout=5) |
There was a problem hiding this comment.
Please don't remove these. They verify there's no regression when someone passes a timeout value
There was a problem hiding this comment.
They have not been removed but moved under "Okay" section below, since those cases did not trigger violations.
| httpx.put('https://gmail.com', verify=True) | ||
| httpx.put('https://gmail.com', verify=False) | ||
| httpx.patch('https://gmail.com', verify=True) | ||
| httpx.patch('https://gmail.com', verify=False) |
There was a problem hiding this comment.
Please just add a separate file rather than abuse this one
There was a problem hiding this comment.
Adding the check for missing timeouts on httpx made this test file report medium violations in the test due to missing timeout values. I did that with the intention of keeping only the violations related to SSL verification disabled. Would you prefer to keep the file as is and bump the number of reported violations here instead?
For the missing timeouts, I can still create a separate test case and example file if you prefer though. I wanted to stay consistent with what has been done in #861, but having a separate test case would probably be preferrable.
Resolves #1059.