[ 
https://issues.apache.org/jira/browse/TS-2237?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14048507#comment-14048507
 ] 

Sudheer Vinukonda commented on TS-2237:
---------------------------------------

Yes, the external URLs should be already encoded - however, internally, I see 
code that decodes the URL strings (e.g. UrlMatcher::Match). So, by the time 
logging receives the URL, it may be in the decoded format and needs to be 
re-encoded.

> 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
>             Fix For: 5.1.0
>
>         Attachments: TS-2237.diff
>
>
> 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.2#6252)

Reply via email to