Le 16/04/2021 à 20:14, Tim Düsterhus a écrit :
Willy,
Christopher,
On 4/16/21 6:56 PM, Willy Tarreau wrote:
Thanks Tim,
The series looks good to me. Except if Willy has any comments, I'll merge it
soon.
No need for me to double-check, I'll trust you (and you know I like
to complain afterwards :-))
I'll probably complain myself afterwards as well :-)
As a heads up I'll definitely add a few more converters (most notably
the percent decoder is missing) and maybe I'll rename the existing ones
in the configuration for consistency once I finish this up.
So: It should definitely considered *EXPERIMENTAL* for 2.4.
Hi Tim,
I'm a bit annoyed by the renaming of existing normalizers. Except if you plan to
do your modification before the release, it is a bad idea to change the name of
a configuration parameter once introduced in a stable release. The feature
itself may be experimental because some bugs are expected but from the
configuration point of view, it should be stable.
However, if you want to wait a bit to finish your work, I can push your patches
in the next branch, pending for the next 2.5. This way, you'll have all the time
to modify it. And because it is a standalone feature, we may plan to backport it
to 2.4 if necessary.
--
Christopher Faulet
