committed two related things to trunk this afternoon: - allow anything if redirecting and no [NE] flag - add another [B] like flag that escapes only controls and spaces.
On Sat, Mar 11, 2023 at 2:30 PM Eric Covener <cove...@gmail.com> wrote: > > Pulling up some of the checks so we can consider the flag: > http://people.apache.org/~covener/patches/rewrite-escaping.diff > > (needs to be duplicated in fixups hook) > > On Fri, Mar 10, 2023 at 11:57 AM Yann Ylavic <ylavic....@gmail.com> wrote: > > > > On Fri, Mar 10, 2023 at 4:34 PM Eric Covener <cove...@gmail.com> wrote: > > > > > > Saw another report on users@ > > > > > > Any thoughts on something like this to just allow spaces? > > > http://people.apache.org/~covener/patches/rewrite-lax.diff > > > > What about: > > > > Index: modules/mappers/mod_rewrite.c > > =================================================================== > > --- modules/mappers/mod_rewrite.c (revision 1908254) > > +++ modules/mappers/mod_rewrite.c (working copy) > > @@ -4814,7 +4814,8 @@ static int hook_uri2file(request_rec *r) > > apr_size_t flen; > > int to_proxyreq; > > > > - if (r->args && *(ap_scan_vchar_obstext(r->args))) { > > + if (rulestatus == ACTION_NOESCAPE > > + && r->args && *(ap_scan_vchar_obstext(r->args))) { > > /* > > * We have a raw control character or a ' ' in r->args. > > * Correct encoding was missed. > > ? > > > > Regards; > > Yann. > > > > -- > Eric Covener > cove...@gmail.com -- Eric Covener cove...@gmail.com
