openstack/swift
Code Issues Proposed changes

Merge "Use path_qs instead of reinventing it"

Browse Source
This commit is contained in:
Zuul 2018-06-15 05:41:15 +00:00 committed by Gerrit Code Review
commit 6274ad49cb
1 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
swift/common/middleware/proxy_logging.py
View File

@ -152,9 +152,6 @@ class ProxyLoggingMiddleware(object):
:param resp_headers: dict of the response headers :param resp_headers: dict of the response headers
""" """
resp_headers = resp_headers or {} resp_headers = resp_headers or {}
the_request = req.path
if req.query_string:
the_request = the_request + '?' + req.query_string
logged_headers = None logged_headers = None
if self.log_hdrs: if self.log_hdrs:
if self.log_hdrs_only: if self.log_hdrs_only:
@ -179,7 +176,7 @@ class ProxyLoggingMiddleware(object):
req.remote_addr, req.remote_addr,
end_gmtime_str, end_gmtime_str,
method, method,
the_request, req.path_qs,
req.environ.get('SERVER_PROTOCOL'), req.environ.get('SERVER_PROTOCOL'),
status_int, status_int,
req.referer, req.referer,