+1.
Sounds reasonable to me.

    On Friday, August 28, 2020, 07:41:18 AM PDT, Brian Neradt 
<brian.ner...@gmail.com> wrote:  
 
 Hi dev@trafficserver.apache.org,

It would be helpful to add HTTP/2 stream and priority support to Traffic
Dump. In order for the plugin to access this information, I propose adding
the following two functions to the TS API:

tsapi TSReturnCode TSHttpTxnClientHttp2StreamIdGet(TSHttpTxn txnp, uint32_t
*stream_id);

tsapi TSReturnCode TSHttpTxnClientHttp2PriorityGet(TSHttpTxn txnp, int
*stream_dependency, int *weight);

I've created a draft PR with this implemented along with Traffic Dump's use
of it:
https://github.com/apache/trafficserver/pull/7149

Please let me know if there are any concerns or suggestions for improvement
concerning this.

Thanks!
Brian

-- 
"Come to Me, all who are weary and heavy-laden, and I will
give you rest. Take My yoke upon you and learn from Me, for
I am gentle and humble in heart, and you will find rest for
your souls. For My yoke is easy and My burden is light."

    ~ Matthew 11:28-30
  

Reply via email to