openstack/swift
Code Issues Proposed changes

Merge "Improve path handling in proxy_logging"

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

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

@ -75,10 +75,10 @@ import sys
import time import time
import six import six
from six.moves.urllib.parse import quote, unquote from six.moves.urllib.parse import quote
from swift.common.swob import Request from swift.common.swob import Request
from swift.common.utils import (get_logger, get_remote_client, from swift.common.utils import (get_logger, get_remote_client,
get_valid_utf8_str, config_true_value, config_true_value,
InputProxy, list_from_csv, get_policy_index) InputProxy, list_from_csv, get_policy_index)
from swift.common.storage_policy import POLICIES from swift.common.storage_policy import POLICIES
@ -152,8 +152,7 @@ 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 {}
req_path = get_valid_utf8_str(req.path) the_request = req.path
the_request = quote(unquote(req_path), QUOTE_SAFE)
if req.query_string: if req.query_string:
the_request = the_request + '?' + req.query_string the_request = the_request + '?' + req.query_string
logged_headers = None logged_headers = None