You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficserver.apache.org by "Nick Kew (Issue Comment Edited) (JIRA)" <ji...@apache.org> on 2011/12/23 20:36:31 UTC

[jira] [Issue Comment Edited] (TS-998) Broken ClientReq in TSAPI

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

Nick Kew edited comment on TS-998 at 12/23/11 7:36 PM:
-------------------------------------------------------

r1221229 fixes the original problem.

Logfile entries are also fixed in my 3.0.1, but are broken in 3.1.1, so there's evidently still a problem.  This breakage to the error log entries is also what induced me to drop attempts to access the data more directly.

Something is calculating a bogus length to the URL to be logged!
                
      was (Author: nick@webthing.com):
    r1221229 fixes the original problem.

Logfile entries are also fixed in my 3.0.1, but are broken in 3.1.1, so there's evidently still a problem.  This breakage to the error log entries is also what induced me to drop attempts to access the data more directly.
                  
> Broken ClientReq in TSAPI
> -------------------------
>
>                 Key: TS-998
>                 URL: https://issues.apache.org/jira/browse/TS-998
>             Project: Traffic Server
>          Issue Type: Bug
>    Affects Versions: 3.0.1
>         Environment: any
>            Reporter: Nick Kew
>            Assignee: Nick Kew
>             Fix For: 3.1.2
>
>
> Extracting a Request using TSHttpTxnClientReqGet API yields a bogus Request line.
> Expected behaviour: In a PRE_REMAP hook it should return the client request line and headers, ideally verbatim.
> Observed behaviour: "http://" is prepended to the request URL:
>   GET /path/ HTTP/1.1
> becomes
>   GET http:///path/ HTTP/1.1
> (yes, that's three slashes)
> Pseudo-code to reproduce from a PRE_REMAP hook:
>   TSHttpTxnClientReqGet(txnp, &buf, &hdr);
>   TSHttpHdrPrint(buf, hdr, iobuf);
>   reader = TSIOBufferReaderAlloc(iobuf);
>   block = TSIOBufferReaderStart(reader);
>   len = TSIOBufferBlockReadAvail(block, reader);
>   data = TSIOBufferBlockReadStart(block, reader, &len);
> Now examine the contents of data.
> Assigned to AMC as suggested yesterday on-list.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira