diff --git a/doc/data/messages/i/invalid-envvar-default/bad.py b/doc/data/messages/i/invalid-envvar-default/bad.py index 9b564b9c8c..f87b0de304 100644 --- a/doc/data/messages/i/invalid-envvar-default/bad.py +++ b/doc/data/messages/i/invalid-envvar-default/bad.py @@ -1,3 +1,4 @@ import os env = os.getenv("SECRET_KEY", 1) # [invalid-envvar-default] +env = os.environ.get("SECRET_KEY", 1) # [invalid-envvar-default] diff --git a/doc/data/messages/i/invalid-envvar-default/good.py b/doc/data/messages/i/invalid-envvar-default/good.py index 103925941e..bcc41cb13c 100644 --- a/doc/data/messages/i/invalid-envvar-default/good.py +++ b/doc/data/messages/i/invalid-envvar-default/good.py @@ -1,3 +1,4 @@ import os env = os.getenv("SECRET_KEY", "1") +env = os.environ.get("SECRET_KEY", "1") diff --git a/doc/whatsnew/fragments/10092.false_negative b/doc/whatsnew/fragments/10092.false_negative new file mode 100644 index 0000000000..764aa756ea --- /dev/null +++ b/doc/whatsnew/fragments/10092.false_negative @@ -0,0 +1,3 @@ +Fix false negative for `invalid-envvar-default` to detect `os.environ.get`. + +Closes #10092 diff --git a/pylint/checkers/stdlib.py b/pylint/checkers/stdlib.py index 9225cd4d26..b9591a61ae 100644 --- a/pylint/checkers/stdlib.py +++ b/pylint/checkers/stdlib.py @@ -30,7 +30,7 @@ THREADING_THREAD = "threading.Thread" COPY_COPY = "copy.copy" OS_ENVIRON = "os._Environ" -ENV_GETTERS = ("os.getenv",) +ENV_GETTERS = ("os.getenv", "os.environ.get") SUBPROCESS_POPEN = "subprocess.Popen" SUBPROCESS_RUN = "subprocess.run" OPEN_MODULE = {"_io", "pathlib", "pathlib._local"}