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

Leif Hedstrom commented on TS-2148:
-----------------------------------

Alan and I took a deeper look, and we think the patch is gtg as is. No need to 
add something to Destroy() since this gets reset in void
HttpSM::_instantiate_func(HttpSM * prototype, HttpSM * new_instance).
                
> handle_cache_operation_on_forward_server_response ignores value of 
> api_server_response_no_store
> -----------------------------------------------------------------------------------------------
>
>                 Key: TS-2148
>                 URL: https://issues.apache.org/jira/browse/TS-2148
>             Project: Traffic Server
>          Issue Type: Bug
>          Components: TS API
>            Reporter: Corey Cossentino
>         Attachments: ts2148_response_no_store.patch
>
>
> In the function 
> {{HttpTransact::handle_cache_operation_on_forward_server_response}} there is 
> a line that checks the value of {{s->api_server_response_no_store}} (line 
> 4162 in HttpTransact.cc). This value will never be true, however, because 
> {{is_response_cacheable}} is called before this check (on line 3934) and sets 
> this value to false (on line 6104).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to