diff options
author | Christian Fueller <christian.fueller@sap.com> | 2018-06-01 13:27:39 +0200 |
---|---|---|
committer | Ashley Camba <ashwoods@gmail.com> | 2018-07-24 14:24:16 +0200 |
commit | f6d79c3bcc25e804b6259fa9c4a6e030f9033bb2 (patch) | |
tree | 014aa2d30e9385b48ebf5e03a79912ccec7b50ce /raven/breadcrumbs.py | |
parent | a698fc406d2e0f26616f0163b2b22e1a3a5b30f5 (diff) | |
download | raven-f6d79c3bcc25e804b6259fa9c4a6e030f9033bb2.tar.gz |
Allow dict-style logging in breadcrumbs processing
Diffstat (limited to 'raven/breadcrumbs.py')
-rw-r--r-- | raven/breadcrumbs.py | 17 |
1 files changed, 14 insertions, 3 deletions
diff --git a/raven/breadcrumbs.py b/raven/breadcrumbs.py index 13987f1..fbf3026 100644 --- a/raven/breadcrumbs.py +++ b/raven/breadcrumbs.py @@ -1,5 +1,6 @@ from __future__ import absolute_import +import collections import os import logging @@ -132,13 +133,23 @@ def _record_log_breadcrumb(logger, level, msg, *args, **kwargs): def processor(data): formatted_msg = msg + format_args = args + + # Extract 'extra' key from kwargs and merge into data + extra = kwargs.pop('extra', {}) + data_value = kwargs + data_value.update(extra) + + if args and len(args) == 1 and isinstance(args[0], collections.Mapping) and args[0]: + format_args = args[0] + data_value.update(format_args) # If people log bad things, this can happen. Then just don't do # anything. try: formatted_msg = text_type(msg) - if args: - formatted_msg = msg % args + if format_args: + formatted_msg = msg % format_args except Exception: pass @@ -151,7 +162,7 @@ def _record_log_breadcrumb(logger, level, msg, *args, **kwargs): 'message': formatted_msg, 'category': logger.name, 'level': logging.getLevelName(level).lower(), - 'data': kwargs, + 'data': data_value, }) record(processor=processor) |