From: Vinay Sajip Date: Sun, 21 Jun 2009 17:37:27 +0000 (+0000) Subject: Issue #6314: logging.basicConfig() performs extra checks on the "level" argument. X-Git-Tag: v2.7a1~923 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=603fb6d66759afe8b5f65f2d456a43e73bb89233;p=python Issue #6314: logging.basicConfig() performs extra checks on the "level" argument. --- diff --git a/Lib/logging/__init__.py b/Lib/logging/__init__.py index 456dec906a..23f7930b69 100644 --- a/Lib/logging/__init__.py +++ b/Lib/logging/__init__.py @@ -1397,6 +1397,10 @@ def basicConfig(**kwargs): root.addHandler(hdlr) level = kwargs.get("level") if level is not None: + if str(level) == level: # If a string was passed, do more checks + if level not in _levelNames: + raise ValueError("Unknown level: %r" % level) + level = _levelNames[level] root.setLevel(level) #--------------------------------------------------------------------------- diff --git a/Misc/NEWS b/Misc/NEWS index 86666ffaa2..2985c3aa70 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -327,7 +327,10 @@ Core and Builtins Library ------- -- Issue #6164: Added an AIX specific linker argument in Distutils +- Issue #6314: logging.basicConfig() performs extra checks on the "level" + argument. + +- Issue #6164: Added an AIX specific linker argument in Distutils unixcompiler. Original patch by Sridhar Ratnakumar. - Issue #6274: Fixed possible file descriptors leak in subprocess.py