Seems reasonable, except maybe `TSHttpTxnPostBufferReaderGet()`? Most of the API that takes a TSHttpTxn is prefixed with 'TSHTTPTxn'.
On Wed, Mar 7, 2018 at 11:15 AM, Bryan Call <[email protected]> wrote: > +1 > > -Bryan > > > > > On Mar 5, 2018, at 11:19 AM, CrazyCow <[email protected]> > wrote: > > > > Hi, Team, > > > > Along with the post buffer PR > > <https://github.com/apache/trafficserver/pull/2335> which buffers post > body > > before opening connections to origin servers, I'd introduce a new API > that > > enables plugins to read the buffered body. > > > > TSIOBufferReader TSIOBufferReaderPostGet(TSHttpTxn txnp); > > > > > > Plugins can read the entire post body via the returned TSIOBufferReader. > > > > > > Please let me know what you think about this new API. Any idea would be > > much appreciated! > > > > > > Thanks, > > Zizhong > >
