On Mon, Mar 13, 2023 at 5:25 PM Eric Covener <cove...@gmail.com> wrote: > > On Mon, Mar 13, 2023 at 12:05 PM Yann Ylavic <ylavic....@gmail.com> wrote: > > > > I could get where you want to with the attached patch (before you > > disabled the test in r1908350). > > It makes so that anything BNEG'ed overrides BCTLS, which can be useful > > to let space through while still encoding controls. > > I had to fix the local_bctls_nospace RewriteRule to not encode '/' > > (not expected in the result) and add a missing '/' in the result too. > > > + RewriteRule ^/modules/rewrite/escaping/local_bctls_nospace/(.*) > /?$1 "[B= /,BNEG,BCTLS]" > + [ "/modules/rewrite/escaping/local_bctls_nospace/foo/bar/%20baz/%0d" > => "foo/bar/ baz/%0d"], # CTLS but allow space > > I'm not sure I understand this, / isn't a CTL so why do we need to > exclude it with B+BNEG to have it passed through in the test?
With BNEG everything not in B= is encoded, so "[B= ?,BNEG]" (regardless of BCTLS) will encode '/'.
