Skip to content

Commit 0e0e4ac

Browse files
miss-islingtonvsajip
authored andcommitted
bpo-39142: Avoid converting namedtuple instances to ConvertingTuple. (GH-17773) (GH-17786)
(cherry picked from commit 46abfc1)
1 parent 88938e4 commit 0e0e4ac

File tree

3 files changed

+37
-1
lines changed

3 files changed

+37
-1
lines changed

Lib/logging/config.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -447,7 +447,7 @@ def convert(self, value):
447447
value = ConvertingList(value)
448448
value.configurator = self
449449
elif not isinstance(value, ConvertingTuple) and\
450-
isinstance(value, tuple):
450+
isinstance(value, tuple) and not hasattr(value, '_fields'):
451451
value = ConvertingTuple(value)
452452
value.configurator = self
453453
elif isinstance(value, str): # str for py3k

Lib/test/test_logging.py

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3223,6 +3223,37 @@ def test_baseconfig(self):
32233223
self.assertRaises(ValueError, bc.convert, 'cfg://!')
32243224
self.assertRaises(KeyError, bc.convert, 'cfg://adict[2]')
32253225

3226+
def test_namedtuple(self):
3227+
# see bpo-39142
3228+
from collections import namedtuple
3229+
3230+
class MyHandler(logging.StreamHandler):
3231+
def __init__(self, resource, *args, **kwargs):
3232+
super().__init__(*args, **kwargs)
3233+
self.resource: namedtuple = resource
3234+
3235+
def emit(self, record):
3236+
record.msg += f' {self.resource.type}'
3237+
return super().emit(record)
3238+
3239+
Resource = namedtuple('Resource', ['type', 'labels'])
3240+
resource = Resource(type='my_type', labels=['a'])
3241+
3242+
config = {
3243+
'version': 1,
3244+
'handlers': {
3245+
'myhandler': {
3246+
'()': MyHandler,
3247+
'resource': resource
3248+
}
3249+
},
3250+
'root': {'level': 'INFO', 'handlers': ['myhandler']},
3251+
}
3252+
with support.captured_stderr() as stderr:
3253+
self.apply_config(config)
3254+
logging.info('some log')
3255+
self.assertEqual(stderr.getvalue(), 'some log my_type\n')
3256+
32263257
class ManagerTest(BaseTest):
32273258
def test_manager_loggerclass(self):
32283259
logged = []
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
A change was made to logging.config.dictConfig to avoid converting instances
2+
of named tuples to ConvertingTuple. It's assumed that named tuples are too
3+
specialised to be treated like ordinary tuples; if a user of named tuples
4+
requires ConvertingTuple functionality, they will have to implement that
5+
themselves in their named tuple class.

0 commit comments

Comments
 (0)