diff options
| author | Milind Changire <mchangir@redhat.com> | 2017-06-26 18:20:23 +0530 |
|---|---|---|
| committer | Aravinda VK <avishwan@redhat.com> | 2017-06-30 07:40:51 +0000 |
| commit | d95535bae2d200c2210feac7568b1fdbf6f545a9 (patch) | |
| tree | 4567521ba582d10099ac4f7722e25986f047d525 /tools/glusterfind | |
| parent | 97defef2375b911c7b6a3924c242ba8ef4593686 (diff) | |
tools/glusterfind: unquote DELETE path before further processing
Problem:
DELETE path is quoted before it reaches glusterfind. This wasn't handled
in the glusterfind code leading to double quoting of path separator
'%2F' to '%252F' i.e. the '%' character in '%2F' itself was quoted to
'%25'
Solution:
unquote the the deleted path before further processing
Change-Id: I2dfbbd7792dc0f9da5c8e02093b0f1c031ff344a
BUG: 1465024
Signed-off-by: Milind Changire <mchangir@redhat.com>
Reviewed-on: https://review.gluster.org/17629
Smoke: Gluster Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Aravinda VK <avishwan@redhat.com>
Diffstat (limited to 'tools/glusterfind')
| -rw-r--r-- | tools/glusterfind/src/changelogdata.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/tools/glusterfind/src/changelogdata.py b/tools/glusterfind/src/changelogdata.py index 0e32d7b7f91..b4a97093aa8 100644 --- a/tools/glusterfind/src/changelogdata.py +++ b/tools/glusterfind/src/changelogdata.py @@ -16,6 +16,7 @@ import os from utils import RecordType from utils import output_path_prepare + class OutputMerger(object): """ Class to merge the output files collected from @@ -421,8 +422,8 @@ class ChangelogData(object): deleted_path = data[4] if len(data) == 5 else "" if deleted_path != "": - deleted_path = output_path_prepare(deleted_path, - self.args) + deleted_path = urllib.unquote_plus(deleted_path.encode("utf-8")) + deleted_path = output_path_prepare(deleted_path, self.args) if self.gfidpath_exists({"gfid": data[1], "type": "NEW", "pgfid1": pgfid1, "bn1": bn1}): |