Issue #8924: logging: Improved error handling for Unicode in exception text.

diff --git a/Lib/logging/__init__.py b/Lib/logging/__init__.py
index 239ae61fadbaeb1adee5a7b4ce05f4c6b3d1b8de..80ec6724e44c3d091e6120a6eeebfa47aecc1907 100644 (file)
--- a/Lib/logging/__init__.py
+++ b/Lib/logging/__init__.py
@@ -473,7 +473,13 @@ class Formatter(object):
         if record.exc_text:
             if s[-1:] != "\n":
                 s = s + "\n"
-            s = s + record.exc_text
+            try:
+                s = s + record.exc_text
+            except UnicodeError:
+                # Sometimes filenames have non-ASCII chars, which can lead
+                # to errors when s is Unicode and record.exc_text is str
+                # See issue 8924
+                s = s + record.exc_text.decode(sys.getfilesystemencoding())
         return s
 
 #

diff --git a/Misc/NEWS b/Misc/NEWS
index 78d8cc70f951f03f0fe98754135a521d5ce1bc87..0d353994ece6e140db2c0949acd16b9d7ed96ad2 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -21,6 +21,7 @@ Core and Builtins
 
 Library
 -------
+- Issue #8924: logging: Improved error handling for Unicode in exception text.
 
 - Issue #8948: cleanup functions and class / module setups and teardowns are
   now honored in unittest debug methods.