On Wed, Aug 7, 2024 at 11:44 AM Ruediger Pluem <rpl...@apache.org> wrote:
>
> On 8/2/24 2:53 AM, yla...@apache.org wrote:
> >
> > --- httpd/httpd/trunk/modules/proxy/mod_proxy_fcgi.c (original)
> > +++ httpd/httpd/trunk/modules/proxy/mod_proxy_fcgi.c Fri Aug 2 00:53:53
> > 2024
>
> > @@ -93,20 +95,40 @@ static int proxy_fcgi_canon(request_rec
> > host = apr_pstrcat(r->pool, "[", host, "]", NULL);
> > }
> >
> > - /* We do not call ap_proxy_canonenc_ex() on the path here because the
> > CGI
> > - * environment variable SCRIPT_FILENAME based on it want the
> > decoded/local
> > - * path, don't let control characters pass still.
> > - *
> > - * XXX: should we encode based on dconf->backend_type though?
> > - */
> > - for (pc = path; *pc; pc++) {
> > - if (apr_iscntrl(*pc)) {
> > + from_handler = apr_table_get(r->notes, "proxy-sethandler") != NULL;
>
> Why do we need the from_handler variable?
Axed in r1921237.
Thanks;
Yann.
