[
https://issues.apache.org/jira/browse/TS-2237?focusedWorklogId=26477&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-26477
]
ASF GitHub Bot logged work on TS-2237:
--------------------------------------
Author: ASF GitHub Bot
Created on: 16/Aug/16 00:18
Start Date: 16/Aug/16 00:18
Worklog Time Spent: 10m
Work Description: Github user maskit commented on the issue:
https://github.com/apache/trafficserver/pull/866
``HttpRequestData::get_string()`` unescapes an URL and it is called from
``UrlMatcher<Data, Result>::Match(RequestData *rdata, Result *result)``. I
think this is the code Sudheer mentioned.
However, at least, the unescaped URL doesn't come out from the function. If
we could assure that no unescaped string flows into the logging system, we will
be able to simply remove some of calls of ``LogUtils::url_escapify``.
Also, I realized that ``TSStringPercentEncode`` uses
``LogUtils::url_escapify`` internally. So changing behavior of
``LogUtils::url_escapify`` would affect plugins.
Issue Time Tracking
-------------------
Worklog Id: (was: 26477)
Time Spent: 2.5h (was: 2h 20m)
> URL encoding wrong in squid.blog
> --------------------------------
>
> Key: TS-2237
> URL: https://issues.apache.org/jira/browse/TS-2237
> Project: Traffic Server
> Issue Type: Bug
> Components: Logging
> Reporter: David Carlin
> Priority: Minor
> Labels: yahoo
> Fix For: sometime
>
> Attachments: TS-2237.diff
>
> Time Spent: 2.5h
> Remaining Estimate: 0h
>
> I was replaying URLs captured from squid.blog and I noticed I was getting
> 404's for some of them when squid.blog showed a 200 for that request. Turns
> out there is an issue with URL encoding. For example:
> Requesting file 'duck%20sports%20authority.gif' via curl will put this in the
> logs:
> duck%2520sports%2520authority.gif
> The % from %20 (space) in the request is being converted to %25 resulting in
> %2520
> I tested both the %<cquc> and %<cquuc> log fields - same thing happens. I
> tested on ATS 3.2.0 and 3.3.5
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
