diff --git a/Lib/logging/__init__.py b/Lib/logging/__init__.py index 9026ef383b1..7b790d2de12 100644 --- a/Lib/logging/__init__.py +++ b/Lib/logging/__init__.py @@ -757,7 +757,7 @@ class StreamHandler(Handler): self.stream.write(fs % msg) else: try: - if hasattr(self.stream, 'encoding'): + if getattr(self.stream, 'encoding', None) is not None: self.stream.write(fs % msg.encode(self.stream.encoding)) else: self.stream.write(fs % msg) diff --git a/Lib/test/test_logging.py b/Lib/test/test_logging.py index 6ed8ca2a43d..5d2b5fd3fd9 100644 --- a/Lib/test/test_logging.py +++ b/Lib/test/test_logging.py @@ -859,6 +859,31 @@ class MemoryTest(BaseTest): ('foo', 'DEBUG', '3'), ]) +class EncodingTest(BaseTest): + def test_encoding_plain_file(self): + # In Python 2.x, a plain file object is treated as having no encoding. + log = logging.getLogger("test") + fn = tempfile.mktemp(".log") + # the non-ascii data we write to the log. + data = "foo\x80" + try: + handler = logging.FileHandler(fn) + log.addHandler(handler) + try: + # write non-ascii data to the log. + log.warning(data) + finally: + log.removeHandler(handler) + handler.close() + # check we wrote exactly those bytes, ignoring trailing \n etc + f = open(fn) + try: + self.failUnlessEqual(f.read().rstrip(), data) + finally: + f.close() + finally: + if os.path.isfile(fn): + os.remove(fn) # Set the locale to the platform-dependent default. I have no idea # why the test does this, but in any case we save the current locale @@ -867,7 +892,8 @@ class MemoryTest(BaseTest): def test_main(): run_unittest(BuiltinLevelsTest, BasicFilterTest, CustomLevelsAndFiltersTest, MemoryHandlerTest, - ConfigFileTest, SocketHandlerTest, MemoryTest) + ConfigFileTest, SocketHandlerTest, MemoryTest, + EncodingTest) if __name__ == "__main__": test_main() diff --git a/Misc/NEWS b/Misc/NEWS index 33866dfa7d1..dbb81101177 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -56,6 +56,8 @@ C-API Library ------- +- Issue #3772: Fixed regression problem in StreamHandler.emit(). + - Issue 600362: Relocated parse_qs() and parse_qsl(), from the cgi module to the urlparse one. Added a PendingDeprecationWarning in the old module, it will be deprecated in the future. @@ -87,7 +89,7 @@ Library - Issue #3708: os.urandom no longer goes into an infinite loop when passed a non-integer floating point number. -- Issue #3110: multiprocessing fails to compiel on solaris 10 due to missing +- Issue #3110: multiprocessing fails to compiel on solaris 10 due to missing SEM_VALUE_MAX. Extension Modules