Re: Multiple times gzip issue and TSHttpTxnHookAdd API

2016-12-01 Thread Meera Mosale Nataraja
On Thu, Dec 1, 2016 at 10:34 AM, Leif Hedstrom wrote: > > > On Nov 29, 2016, at 7:57 PM, James Peach wrote: > > > >> > >> On Nov 29, 2016, at 11:38 AM, Meera Mosale Nataraja > wrote: > >> > >> Hello all, > >> > >> I'm working on TS-5024 > where >

Re: Multiple times gzip issue and TSHttpTxnHookAdd API

2016-12-01 Thread Leif Hedstrom
> On Nov 29, 2016, at 7:57 PM, James Peach wrote: > >> >> On Nov 29, 2016, at 11:38 AM, Meera Mosale Nataraja >> wrote: >> >> Hello all, >> >> I'm working on TS-5024 where >> the content is gzip’ed multiple times. Please find the sample output

Re: Multiple times gzip issue and TSHttpTxnHookAdd API

2016-11-29 Thread James Peach
> On Nov 29, 2016, at 11:38 AM, Meera Mosale Nataraja wrote: > > Hello all, > > I'm working on TS-5024 where > the content is gzip’ed multiple times. Please find the sample output > provided below which indicates multiple gzips. > > curl -v -o/de

Re: Multiple times gzip issue and TSHttpTxnHookAdd API

2016-11-29 Thread Sudheer Vinukonda
as long as Meera's solution doesn't break calling the same hook multiple times on the same Cont/Txn, I agree it seems fine :) Thanks, Sudheer From: Leif Hedstrom To: dev@trafficserver.apache.org; Sudheer Vinukonda Sent: Tuesday, November 29, 2016 2:33 PM Subject: Re: Multiple time

Re: Multiple times gzip issue and TSHttpTxnHookAdd API

2016-11-29 Thread Leif Hedstrom
> On Nov 29, 2016, at 1:53 PM, Sudheer Vinukonda > wrote: > > I think there are some plugins that do depend on a hook getting called > multiple times (example, redirect response based plugins like escalate, > collapsed_forwarding etc). This restriction is per transaction, I couldn't think of

Re: Multiple times gzip issue and TSHttpTxnHookAdd API

2016-11-29 Thread vijay mamidi
I think the patch is going to restrict plugin registering to the same hook multiple times but i don't think it will restrict number of times a hook is called. -Vijay On Tue, Nov 29, 2016 at 12:53 PM, Sudheer Vinukonda < sudheervinuko...@yahoo.com.invalid> wrote: > I think there are some plugins

Re: Multiple times gzip issue and TSHttpTxnHookAdd API

2016-11-29 Thread Sudheer Vinukonda
I think there are some plugins that do depend on a hook getting called multiple times (example, redirect response based plugins like escalate, collapsed_forwarding etc). To solve the issue of multiple gzips, would it be simpler to store in the continuation the info that gzip has already been co