Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

No need to check httpx client without timeout defined #1177

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

No need to check httpx client without timeout defined #1177

wants to merge 2 commits into from

Commits on Sep 27, 2024

  1. No need to check httpx client without timeout defined 

    Unlike python-requests, the httpx client has a default
timeout of 5 seconds on its class and functions. As such,
there is no need for Bandit to check for an undefined
timeout. However, explicitly setting the timeout to None
is still a potential problem as that would create a
situtation where the client would block forever.

Fixes: PyCQA#1175

Signed-off-by: Eric Brown <[email protected]>
    @ericwb
    ericwb committed Sep 27, 2024
    Configuration menu
    Copy the full SHA
    9b958d1 View commit details
    Browse the repository at this point in the history

Commits on Oct 1, 2024

  1. Merge branch 'main' into httpx_no_timeout

    @ericwb
    ericwb authored Oct 1, 2024
    Configuration menu
    Copy the full SHA
    3d61d52 View commit details
    Browse the repository at this point in the history