summary refs log tree commit diff
path: root/synapse/util/logformatter.py
diff options
context:
space:
mode:
authorRichard van der Hoff <richard@matrix.org>2017-10-11 17:26:17 +0100
committerRichard van der Hoff <richard@matrix.org>2017-10-11 17:26:17 +0100
commitf30c4ed2bc2255dc7182bd026fb6437afec735a5 (patch)
treea4bd0c384afc257d0b811f2b1e060b5d1f6e74c9 /synapse/util/logformatter.py
parentFancy logformatter to format exceptions better (diff)
downloadsynapse-f30c4ed2bc2255dc7182bd026fb6437afec735a5.tar.xz
logformatter: fix AttributeError
make sure we have the relevant fields before we try to log them.
Diffstat (limited to 'synapse/util/logformatter.py')
-rw-r--r--synapse/util/logformatter.py14
1 files changed, 11 insertions, 3 deletions
diff --git a/synapse/util/logformatter.py b/synapse/util/logformatter.py

index 60504162e9..cdbc4bffd7 100644 --- a/synapse/util/logformatter.py +++ b/synapse/util/logformatter.py
@@ -33,9 +33,17 @@ class LogFormatter(logging.Formatter): def formatException(self, ei): sio = StringIO.StringIO() - sio.write("Capture point (most recent call last):\n") - traceback.print_stack(ei[2].tb_frame.f_back, None, sio) - traceback.print_exception(ei[0], ei[1], ei[2], None, sio) + (typ, val, tb) = ei + + # log the stack above the exception capture point if possible, but + # check that we actually have an f_back attribute to work around + # https://twistedmatrix.com/trac/ticket/9305 + + if tb and hasattr(tb.tb_frame, 'f_back'): + sio.write("Capture point (most recent call last):\n") + traceback.print_stack(tb.tb_frame.f_back, None, sio) + + traceback.print_exception(typ, val, tb, None, sio) s = sio.getvalue() sio.close() if s[-1:] == "\n":