infrastructure/ansible
1
0
Fork 0
Code Pull requests 7 Activity Actions

Ralph advised to browse top-down and adjust doc

Browse source
This commit is contained in:
Pierre-Yves Chibon 2016-06-23 16:22:04 +02:00
parent f42cbb3d88
commit 93c3140826
1 changed files with 5 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

14
roles/fedmsg/base/templates/logging.py.j2
View file

@ -98,19 +98,15 @@ class ContextInjector(logging.Filter):
This is insane. This is insane.
Unless your method or function is passed a reference to the trac There is no way to know the actual fedmsg message at this point in
'request' object, there is no way to get ahold of the currently the code, so we're crawling our way down the call stack until we
logged in user. Furthermore, there is no way globally to get ahold find the first place with a 'msg' local instance variable and
of the current request object. attempt to extract the msg_id from it (later).
Here, we crawl our way back up the call stack until we find the
first place that has 'req' as a local instance variable and attempt
to extract the username of the current user from that.
Please forgive me (and Ralph, the original author of this code). Please forgive me (and Ralph, the original author of this code).
""" """
for frame in (f[0] for f in inspect.stack()): for frame in (f[0] for f in reversed(inspect.stack())):
if 'msg' in frame.f_locals: if 'msg' in frame.f_locals:
return frame.f_locals['msg'] return frame.f_locals['msg']