Re: [PROPOSAL] Deprecate the TSHttpTxnFollowRedirect() API

2018-02-06 Thread Steven R. Feltner
+1 On 2/6/18, 1:03 PM, "Alan Carroll" wrote: +1 On Tue, Feb 6, 2018 at 7:10 AM, Bryan Call wrote: > +1 > > -Bryan > > > On Feb 5, 2018, at 11:02 PM, Leif Hedstrom wrote: > > > > We’ve made the following configuration properly overridable as of 8.0

Re: [PROPOSAL] Deprecate the TSHttpTxnFollowRedirect() API

2018-02-06 Thread vijay mamidi
+1 > On Feb 6, 2018, at 9:03 AM, Alan Carroll > wrote: > > +1 > >> On Tue, Feb 6, 2018 at 7:10 AM, Bryan Call wrote: >> >> +1 >> >> -Bryan >> >>> On Feb 5, 2018, at 11:02 PM, Leif Hedstrom wrote: >>> >>> We’ve made the following configuration properly overridable as of 8.0.0: >>> >>>

Re: [PROPOSAL] Deprecate the TSHttpTxnFollowRedirect() API

2018-02-06 Thread Alan Carroll
+1 On Tue, Feb 6, 2018 at 7:10 AM, Bryan Call wrote: > +1 > > -Bryan > > > On Feb 5, 2018, at 11:02 PM, Leif Hedstrom wrote: > > > > We’ve made the following configuration properly overridable as of 8.0.0: > > > > CONFIG proxy.config.http.number_of_redirections INT 0 > > > > > > I feel th

Re: [PROPOSAL] Deprecate the TSHttpTxnFollowRedirect() API

2018-02-06 Thread Bryan Call
+1 -Bryan > On Feb 5, 2018, at 11:02 PM, Leif Hedstrom wrote: > > We’ve made the following configuration properly overridable as of 8.0.0: > > CONFIG proxy.config.http.number_of_redirections INT 0 > > > I feel that the API for TSHttpTxnFollowRedirect() is now superfluous, and the > im

[PROPOSAL] Deprecate the TSHttpTxnFollowRedirect() API

2018-02-05 Thread Leif Hedstrom
We’ve made the following configuration properly overridable as of 8.0.0: CONFIG proxy.config.http.number_of_redirections INT 0 I feel that the API for TSHttpTxnFollowRedirect() is now superfluous, and the implementation of it got a little weird. So my proposal is that we mark this depr