You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@trafficserver.apache.org by "James Peach (JIRA)" <ji...@apache.org> on 2013/08/23 20:03:52 UTC

[jira] [Commented] (TS-2148) handle_cache_operation_on_forward_server_response ignores value of api_server_response_no_store

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

James Peach commented on TS-2148:
---------------------------------

Reviewed. I think this is a good patch.
                
> 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