Re: [API] Change TSHttpArgs to split reserved indices for TXN and SSN args.

2017-12-04 Thread Bryan Call
+1 - It is a good idea to separate them and have it consistent across older and newer APIs. -Bryan > On Dec 1, 2017, at 2:39 PM, Alan Carroll > wrote: > > As discussed in issue #2883, based on work in issue #2388, I would like to > split the TXN and SSN plugin argument index reservations.

Re: [API] Change TSHttpArgs to split reserved indices for TXN and SSN args.

2017-12-03 Thread Alan Carroll
No. This would be a breaking API change for 8.0. On Sun, Dec 3, 2017 at 3:44 PM, James Peach wrote: > > > > On Dec 1, 2017, at 2:39 PM, Alan Carroll > wrote: > > > > As discussed in issue #2883, based on work in issue #2388, I would like > to split the TXN and SSN plugin argument index reservat

Re: [API] Change TSHttpArgs to split reserved indices for TXN and SSN args.

2017-12-03 Thread James Peach
> On Dec 1, 2017, at 2:39 PM, Alan Carroll > wrote: > > As discussed in issue #2883, based on work in issue #2388, I would like to > split the TXN and SSN plugin argument index reservations. Currently nothing > in the TS code base uses SSN plugin args therefore this change will not > affect

Re: [API] Change TSHttpArgs to split reserved indices for TXN and SSN args.

2017-12-03 Thread Leif Hedstrom
> On Dec 1, 2017, at 3:39 PM, Alan Carroll > wrote: > > As discussed in issue #2883, based on work in issue #2388, I would like to > split the TXN and SSN plugin argument index reservations. Currently nothing > in the TS code base uses SSN plugin args therefore this change will not > affect

[API] Change TSHttpArgs to split reserved indices for TXN and SSN args.

2017-12-01 Thread Alan Carroll
As discussed in issue #2883, based on work in issue #2388, I would like to split the TXN and SSN plugin argument index reservations. Currently nothing in the TS code base uses SSN plugin args therefore this change will not affect any example, production, or experimental plugin provided by the code