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.

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

Pylint missing-timeout / W3101 is a new exception #32

Open
ross-spencer opened this issue Nov 2, 2022 · 0 comments
Open

Pylint missing-timeout / W3101 is a new exception #32

ross-spencer opened this issue Nov 2, 2022 · 0 comments
Labels
continuous-integration Issues related to continuous integration

Comments

@ross-spencer
Copy link
Member

Pylint have introduced a new exception, W3101 which is currently in our ignore patterns. We should determine what the best course of action is, whether there is a default timeout that is reasonable for our requests, and implement it in code.

missing-timeout / W3101
Message emitted:

Missing timeout argument for method '%s' can cause your program to hang 
indefinitely

Description:

Used when a method needs a 'timeout' parameter in order to avoid waiting for a 
long time. If no timeout is specified explicitly the default value is used. 
For example for 'requests' the program will never time out (i.e. hang 
indefinitely).

See: https://pylint.pycqa.org/en/latest/user_guide/messages/warning/missing-timeout.html

@ross-spencer ross-spencer added the continuous-integration Issues related to continuous integration label Nov 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
continuous-integration Issues related to continuous integration
Projects
None yet
Development

No branches or pull requests

1 participant