+1 for removing it for 11.0. Apparently the operator is from the initial commit. We should have deprecated the operator when we added set-config operator.
-- Masakazu On Wed, May 7, 2025 at 4:18 PM Leif Hedstrom <zw...@apache.org> wrote: > Well, dangit, it turns out, you could use this functionality if you > figured out the secret operator name to use (not the one from the > documentation). > > So, albeit not working as documented, it is possible for someone to use or > have used this functionality, ignoring the broken docs and using the > source. So, probably a bad idea to remove this feature in 10.2 at least, > but maybe a consideration for 11.0. > > — Leif > > > > On May 7, 2025, at 11:06 AM, Leif Hedstrom <zw...@apache.org> wrote: > > > > Hi all, > > > > I’d like to remove the operator set-timeout-out from the header_rewrite > plugin for ATS v10.2.0. Yes, it is technically an incompatible change, but > it turns out that the implementation of this operator just doesn’t work at > all (you get an error if you try to use it). It could be fixed, but I feel > that it’s superfluous and better done via e.g. > > > > set-config proxy.config.http.transaction_no_activity_timeout_out 120 > > > > > > I’ll send a separate email, but I also think we should deprecate the > underlying APIs and HttpSM support for this in the InkAPIs for v10.2, and > remove them for v11.0. They just have no purpose since we made all this > overridable configurations. > > > > If no one objects, I’ll make this change to header_rewrite this week. > > > > Cheers, > > > > — Leif > > > >
