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

Sudheer Vinukonda edited comment on TS-3500 at 4/9/15 9:20 PM:
---------------------------------------------------------------

Fully agree - fwiw, it's not entirely un-precedented though (like most things 
with ATS :=)). There are other Get() API that seem to also do things that are 
not very clean (e.g. {{TSHttpTxnCachedReqGet}}, {{TSHttpTxnCachedRespGet}}, 
{{TSHttpTxnCacheLookupUrlGet}}).

Btw, the copy (second patch) I was referring to was not from within the API :) 
- no change in the API, but, just create/copy the pristine_url explicitly when 
remap is skipped. In any case, it's all good now, with a combination of the two 
:)


was (Author: sudheerv):
Fully agree - fwiw, it's not entirely un-precedented though (like most things 
with ATS :=)). There are other Get() API that seem to also do things that are 
not very clean (e.g. {{TSHttpTxnCachedReqGet}}, {{TSHttpTxnCachedRespGet}}, 
{{TSHttpTxnCacheLookupUrlGet}}).

> pristine url is not populated when TSSkipRemappingSet is enabled
> ----------------------------------------------------------------
>
>                 Key: TS-3500
>                 URL: https://issues.apache.org/jira/browse/TS-3500
>             Project: Traffic Server
>          Issue Type: Bug
>          Components: Core
>            Reporter: Sudheer Vinukonda
>            Assignee: Sudheer Vinukonda
>             Fix For: 6.0.0
>
>
> pristine url is not populated when TSSkipRemappingSet is enabled



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to