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

ASF GitHub Bot logged work on TS-2237:
--------------------------------------

                Author: ASF GitHub Bot
            Created on: 18/Aug/16 00:58
            Start Date: 18/Aug/16 00:58
    Worklog Time Spent: 10m 
      Work Description: Github user maskit commented on the issue:

    https://github.com/apache/trafficserver/pull/866
  
    I'd say it is not sufficient.
    
    As a workaround for most cases, it works, and personally I'm OK with it as 
is. However, I think the behavior of `TSStringPercentEncode` shouldn't be 
changed because it's just a workaround for logging issue.
    
    So, if the API keeps current behavior, then I'm fine with landing this 
change.


Issue Time Tracking
-------------------

    Worklog Id:     (was: 26560)
    Time Spent: 2h 50m  (was: 2h 40m)

> 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
>            Assignee: Sudheer Vinukonda
>            Priority: Minor
>              Labels: yahoo
>             Fix For: 7.0.0
>
>         Attachments: TS-2237.diff
>
>          Time Spent: 2h 50m
>  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)

Reply via email to