trafficserver-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From James Peach <>
Subject Re: TS-4703 && PR-829
Date Thu, 28 Jul 2016 10:05:55 GMT

> On Jul 28, 2016, at 7:33 PM, James Peach <> wrote:
>> On Jul 28, 2016, at 5:50 AM, Petar Bozhidarov Penkov <>
>> Hello,
>> I am writing in accordance with the referenced Pull Request and JIRA issue.
>> I am proposing a GET-er method for Transactions's underlying protocol. This
>> relates to TS-2987 and is effectively one of the proposed solutions, a
>> wrapper around ProxyClientTransaction::get_protocol_string() . The proposed
>> API is as follows:
>> *tsapi const char *TSHttpTxnClientProtocolGet(TSHttpTxn txnp);*
>> **name credit goes to Susan Hinrichs and Alan Carroll*
> Hi Petar,
> I’m not sure that this is the right approach. get_protocol_string() simply distinguishes
HTTP/2 sessions, so it it not something that I think is ready to become API. Since we already
have an abstraction for HTTP versions (see TSHttpHdrVersionGet), maybe a better approach is
to expose a convenience API that returns the transaction HTTP version as a TS_HTTP_VERSION()

We also have a pending review for TSHttpTxnIsWebsocket(), which seems to overlap with this

>> The reason I believe this change is necessary is that plugins do not have
>> any clean way to access this data
> What data are you looking for? Whether the transaction is part of a HTTP/2 session? Anything
>> ( I am not sure if there is any at all)
>> and for one of the plugins I am working on it is crucial that this
>> information is logged. A specific example would be a plugin that hooks on
>> TXN_START and logs the protocol string.This function provides a simple
>> abstraction over the underlying method, it is as simple to use as it can
>> get, and therefore I believe it is a reasonable solution to the problem.
>> JIRA:
>> PR:
>> Thank you for your time and consideration! Please let me know if you have
>> any questions.
>> Yours Sincerely,
>> Petar Penkov

View raw message