From: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com>
Date: Tue, 7 May 2019 21:36:39 +0000 (+0200)
Subject: bpo-36015: Handle StreamHandler representaton of stream with an integer name (GH... 
X-Git-Tag: v3.8.0b1~448
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=ca87eebb22d202c33f3317cbf85059cadc64fa9f;p=python

bpo-36015: Handle StreamHandler representaton of stream with an integer name (GH-11908)
---

diff --git a/Lib/logging/__init__.py b/Lib/logging/__init__.py
index 07a0c0c4ae..16812ec8d5 100644
--- a/Lib/logging/__init__.py
+++ b/Lib/logging/__init__.py
@@ -1111,6 +1111,8 @@ class StreamHandler(Handler):
     def __repr__(self):
         level = getLevelName(self.level)
         name = getattr(self.stream, 'name', '')
+        #  bpo-36015: name can be an int
+        name = str(name)
         if name:
             name += ' '
         return '<%s %s(%s)>' % (self.__class__.__name__, name, level)
diff --git a/Lib/test/test_logging.py b/Lib/test/test_logging.py
index 950217cec2..bc99c3adbe 100644
--- a/Lib/test/test_logging.py
+++ b/Lib/test/test_logging.py
@@ -760,6 +760,10 @@ class TestStreamHandler(logging.StreamHandler):
     def handleError(self, record):
         self.error_record = record
 
+class StreamWithIntName(object):
+    level = logging.NOTSET
+    name = 2
+
 class StreamHandlerTest(BaseTest):
     def test_error_handling(self):
         h = TestStreamHandler(BadStream())
@@ -797,6 +801,10 @@ class StreamHandlerTest(BaseTest):
         actual = h.setStream(old)
         self.assertIsNone(actual)
 
+    def test_can_represent_stream_with_int_name(self):
+        h = logging.StreamHandler(StreamWithIntName())
+        self.assertEqual(repr(h), '<StreamHandler 2 (NOTSET)>')
+
 # -- The following section could be moved into a server_helper.py module
 # -- if it proves to be of wider utility than just test_logging