[Python-checkins] bpo-31457: Don't omit inner ``process()`` calls with nested LogAdapters (GH-4044) (#6568)

Łukasz Langa webhook-mailer at python.org
Sun Apr 22 18:00:33 EDT 2018


https://github.com/python/cpython/commit/26c289dd76ed07dbf37494bfa483c525f6828bd9
commit: 26c289dd76ed07dbf37494bfa483c525f6828bd9
branch: 3.6
author: Miss Islington (bot) <31488909+miss-islington at users.noreply.github.com>
committer: Łukasz Langa <lukasz at langa.pl>
date: 2018-04-22T15:00:30-07:00
summary:

bpo-31457: Don't omit inner ``process()`` calls with nested LogAdapters (GH-4044) (#6568)

This used to be the case on Python 2.  Commit
212b590e118e3650b596917021ed9612a918180b changed the implementation for Python
3, making the `log()` method of LogAdapter call `logger._log()` directly.  This
makes nested log adapters not execute their ``process()`` method.  This patch
fixes the issue.

Also, now proxying `name`, too, to make `repr()` work with nested log adapters.

New tests added.
(cherry picked from commit ce9e62544571e7ade7186697d5dd065fb4c5243f)

Co-authored-by: Łukasz Langa <lukasz at langa.pl>

files:
A Misc/NEWS.d/next/Library/2017-10-18-19-05-17.bpo-31457.KlE6r8.rst

diff --git a/Misc/NEWS.d/next/Library/2017-10-18-19-05-17.bpo-31457.KlE6r8.rst b/Misc/NEWS.d/next/Library/2017-10-18-19-05-17.bpo-31457.KlE6r8.rst
new file mode 100644
index 000000000000..49463c4fd15b
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2017-10-18-19-05-17.bpo-31457.KlE6r8.rst
@@ -0,0 +1,2 @@
+If nested log adapters are used, the inner ``process()`` methods are no
+longer omitted.



More information about the Python-checkins mailing list