[jira] [Commented] (TS-3189) Do not start reading data on server to user agent tunnel too soon

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

ASF GitHub Bot commented on TS-3189:
------------------------------------

GitHub user shinrich opened a pull request:

    https://github.com/apache/trafficserver/pull/143

    TS-3189: Delay initial do_io_read

    Delay the do_io_read on the server to user agent tunnel to avoid cases of the
    incorrect tunnel handling EOS in the post case.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/shinrich/trafficserver ts-3189

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/trafficserver/pull/143.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #143
    
----
commit 2646c3d631b4d98c386582442bdecd47666a341a
Author: shinrich <sh...@network-geographics.com>
Date:   2014-11-11T17:24:13Z

    TS-3189
    Delay the do_io_read on the server to user agent tunnel to avoid cases of the
    incorrect tunnel handling EOS in the post case.

----


> Do not start reading data on server to user agent tunnel too soon
> -----------------------------------------------------------------
>
>                 Key: TS-3189
>                 URL: https://issues.apache.org/jira/browse/TS-3189
>             Project: Traffic Server
>          Issue Type: Bug
>            Reporter: Susan Hinrichs
>            Assignee: Susan Hinrichs
>
> The original logic would set up a do_io_read for max_int in HttpSM::attach_server_session.
> However, at this point there still may be things finishing up in the user_agent to server tunnel (for posts).  We were seeing occasional cases of the EOS for the tunnel to user_agent communication being incorrectly delivered to the consumer of the user_agent to server tunnel.
> It is sufficient to set up a 0 length read in HttpSM::attach_server_session.  This will enable the correct handlers to deal with inactivity timeouts.  Then we can setup the real read in HttpSM::setup_server_read_response_header() after we know that the user_agent to origin server tunnel has been taken down. 



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