trafficserver pull request: Fix an further keepalive issue when responding ...

[finaldie <gi...@git.apache.org>]

GitHub user finaldie opened a pull request:

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

    Fix an further keepalive issue when responding with 204

    Just found there is a further keepalive issue when responding with 204 return code, when I emit a curl call, I always get the `Connection: close` and `* Closing connection 0`, the result:
    
    ```
    $ curl -v -X POST --data-binary @test.json 'http://localhost/v0/test'
    * About to connect() to localhost port 80 (#0)
    *   Trying ::1...
    * Adding handle: conn: 0x12d8c40
    * Adding handle: send: 0
    * Adding handle: recv: 0
    * Curl_addHandleToPipeline: length: 1
    * - Conn 0 (0x12d8c40) send_pipe: 1, recv_pipe: 0
    * Connection refused
    *   Trying 127.0.0.1...
    * Connected to localhost (127.0.0.1) port 80 (#0)
    > POST /v0/test HTTP/1.1
    > User-Agent: curl/7.33.0
    > Host: localhost
    > Accept: */*
    > Content-Length: 1560
    > Content-Type: application/x-www-form-urlencoded
    > Expect: 100-continue
    >
    * Done waiting for 100-continue
    < HTTP/1.1 204 No Content
    < Date: Thu, 19 Dec 2013 12:18:07 GMT
    < Content-Length: 0
    < Cache-Control: private
    < Content-Type: application/json
    < Age: 1
    < Connection: close
    * Server ATS/4.0.2 is not blacklisted
    < Server: ATS/4.0.2
    <
    * Closing connection 0
    ```
    
    The problem is that the trafficserver check the request content length, not the response.
    To solve it, we need to check the content length from origin server's response.
    After apply the patch, the response returned with `Connection: keep-alive` while curl show the `* Connection #0 to host localhost left intact`, the detail as following:
    
    ```
    $ curl -v -X POST --data-binary @test.json 'http://localhost/v0/test'
    * About to connect() to localhost port 80 (#0)
    *   Trying ::1...
    * Adding handle: conn: 0x1323c40
    * Adding handle: send: 0
    * Adding handle: recv: 0
    * Curl_addHandleToPipeline: length: 1
    * - Conn 0 (0x1323c40) send_pipe: 1, recv_pipe: 0
    * Connection refused
    *   Trying 127.0.0.1...
    * Connected to localhost (127.0.0.1) port 80 (#0)
    > POST /v0/test HTTP/1.1
    > User-Agent: curl/7.33.0
    > Host: localhost
    > Accept: */*
    > Content-Length: 1560
    > Content-Type: application/x-www-form-urlencoded
    > Expect: 100-continue
    >
    * Done waiting for 100-continue
    < HTTP/1.1 204 No Content
    < Date: Thu, 19 Dec 2013 12:49:13 GMT
    < Content-Length: 0
    < Cache-Control: private
    < Content-Type: application/json
    < Age: 1
    < Connection: keep-alive
    * Server ATS/4.0.2 is not blacklisted
    < Server: ATS/4.0.2
    <
    * Connection #0 to host localhost left intact
    ```
    
    Thanks,
    Yuzhang

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

    $ git pull https://github.com/finaldie/trafficserver master

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

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

----
commit c4748069b9309ace155e0c73da0c9441784a4773
Author: finaldie <hy...@gmail.com>
Date:   2013-12-19T15:54:22Z

    Fix an further keepalive issue when responding with 204

----