On Thu, Aug 8, 2024 at 11:45 AM Ruediger Pluem <rpl...@apache.org> wrote:
>
> On 7/27/24 3:54 PM, yla...@apache.org wrote:
> >
> > --- httpd/httpd/trunk/modules/proxy/mod_proxy_fcgi.c (original)
> > +++ httpd/httpd/trunk/modules/proxy/mod_proxy_fcgi.c Sat Jul 27 13:54:08
> > 2024
> > @@ -133,7 +133,6 @@ static int proxy_fcgi_canon(request_rec
> > /* It has to be on disk for this to work */
> > if (!strcasecmp(pathinfo_type, "full")) {
> > rconf->need_dirwalk = 1;
> > - ap_unescape_url_keep2f(path, 0);
> > }
> > else if (!strcasecmp(pathinfo_type, "first-dot")) {
> > char *split = ap_strchr(path, '.');
> > @@ -348,10 +347,11 @@ static apr_status_t send_environment(pro
> > fcgi_req_config_t *rconf = ap_get_module_config(r->request_config,
> > &proxy_fcgi_module);
> > fcgi_dirconf_t *dconf = ap_get_module_config(r->per_dir_config,
> > &proxy_fcgi_module);
> >
> > - if (rconf) {
> > - if (rconf->need_dirwalk) {
> > - ap_directory_walk(r);
> > - }
> > + if (rconf && rconf->need_dirwalk) {
> > + char *saved_filename = r->filename;
> > + r->filename = r->uri;
>
> Why not using the result of the below Strip proxy: prefixes for r->filename?
> In case of a Rewriterule I guess r->uri and
> r->filename could be fundamentally different path wise.
Did this in r1921238, as well as setting up the expected request
fields for ap_directory_walk().
Regards;
Yann.
