launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #26167
Re: [Merge] ~pappacena/launchpad:gunicorn-appserver-tests into launchpad:master
Diff comments:
> diff --git a/lib/lp/services/webapp/publication.py b/lib/lp/services/webapp/publication.py
> index 7325d9f..b245c92 100644
> --- a/lib/lp/services/webapp/publication.py
> +++ b/lib/lp/services/webapp/publication.py
> @@ -871,6 +872,8 @@ def tracelog(request, prefix, msg):
> easier. ``prefix`` should be unique and contain no spaces, and
> preferably a single character to save space.
> """
> - tracelog = ITraceLog(request, None)
> - if tracelog is not None:
> - tracelog.log('%s %s' % (prefix, six.ensure_str(msg, 'US-ASCII')))
> + if not config.use_gunicorn:
> + msg = '%s %s' % (prefix, six.ensure_str(msg, 'US-ASCII'))
> + tracelog = ITraceLog(request, None)
> + if tracelog is not None:
> + tracelog.log(msg)
Ok! Adding a DEBUG log here.
--
https://code.launchpad.net/~pappacena/launchpad/+git/launchpad/+merge/396691
Your team Launchpad code reviewers is subscribed to branch ~pappacena/launchpad:gunicorn-appserver-tests.
References