Commit 8a435c47 authored by Vinay Sajip's avatar Vinay Sajip

Fixed #9310: backported logging fix from 2.7.

parent ae7dd8fa
...@@ -104,6 +104,9 @@ def _resolve(name): ...@@ -104,6 +104,9 @@ def _resolve(name):
def _strip_spaces(alist): def _strip_spaces(alist):
return map(lambda x: string.strip(x), alist) return map(lambda x: string.strip(x), alist)
def _encoded(s):
return s if isinstance(s, str) else s.encode('utf-8')
def _create_formatters(cp): def _create_formatters(cp):
"""Create and return formatters""" """Create and return formatters"""
flist = cp.get("formatters", "keys") flist = cp.get("formatters", "keys")
...@@ -214,7 +217,7 @@ def _install_loggers(cp, handlers, disable_existing_loggers): ...@@ -214,7 +217,7 @@ def _install_loggers(cp, handlers, disable_existing_loggers):
#avoid disabling child loggers of explicitly #avoid disabling child loggers of explicitly
#named loggers. With a sorted list it is easier #named loggers. With a sorted list it is easier
#to find the child loggers. #to find the child loggers.
existing.sort() existing.sort(key=_encoded)
#We'll keep the list of existing loggers #We'll keep the list of existing loggers
#which are children of named loggers here... #which are children of named loggers here...
child_loggers = [] child_loggers = []
......
...@@ -69,6 +69,12 @@ class BaseTest(unittest.TestCase): ...@@ -69,6 +69,12 @@ class BaseTest(unittest.TestCase):
finally: finally:
logging._releaseLock() logging._releaseLock()
# Set two unused loggers: one non-ASCII and one Unicode.
# This is to test correct operation when sorting existing
# loggers in the configuration code. See issues 8201, 9310.
logging.getLogger("\xab\xd7\xbb")
logging.getLogger(u"\u013f\u00d6\u0047")
self.root_logger = logging.getLogger("") self.root_logger = logging.getLogger("")
self.original_logging_level = self.root_logger.getEffectiveLevel() self.original_logging_level = self.root_logger.getEffectiveLevel()
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment