You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficserver.apache.org by "Susan Hinrichs (JIRA)" <ji...@apache.org> on 2015/09/11 22:34:45 UTC

[jira] [Commented] (TS-3905) proxy.config.http.keep_alive_no_activity_timeout_out is not used

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

Susan Hinrichs commented on TS-3905:
------------------------------------

Actually as I look into get_inactivity_timeout some more, I question the validity of the code above in all cases.  The three instances of that line in HttpSessionManager.cc should be reviewed.

> proxy.config.http.keep_alive_no_activity_timeout_out is not used
> ----------------------------------------------------------------
>
>                 Key: TS-3905
>                 URL: https://issues.apache.org/jira/browse/TS-3905
>             Project: Traffic Server
>          Issue Type: Bug
>          Components: HTTP
>            Reporter: Susan Hinrichs
>            Assignee: Susan Hinrichs
>
> The keep_alive_no_activity_timeout_in is set correctly on the HttpClientSession when the transaction releases it.  The client session is then hanging out until the next transaction appears, and the keep_alive_no_activity_timeout_in should apply instead of the transaction_no_activity_timeout_in.
> For the server session side, the keep_alive_no_activity_timeout_out and transaction_no_activity_timeout_out should apply.  The keep_alive_no_activity_timeout_out does get set correctly when the server session is attached to the client session to timeout via the HttpClientSession::attach_server_session_method().
> But in ServerSessionPool::releaseSession, the following is called
> {code}
> ss->get_netvc()->set_inactivity_timeout(ss->get_netvc()->get_inactivity_timeout());
> {code}
> My reading is that this will reset the inactivity timeout of the server session to whatever it was last set to.  Instead it should set the inactivity timeout to keep_alive_no_activity_timeout_out.



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