Fwd: [Bug 63437] MergeSlashes option breaks protocol specifier in URIs

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Fwd: [Bug 63437] MergeSlashes option breaks protocol specifier in URIs

Eric Covener
Any thoughts on this one?  We could leave the r->uri untouched for fwd
proxy requests.
Or, just wait, since this is the only report and it's technically
hitting the private module.

---------- Forwarded message ---------
From: <[hidden email]>
Date: Wed, May 15, 2019 at 11:05 AM
Subject: [Bug 63437] MergeSlashes option breaks protocol specifier in URIs
To: <[hidden email]>


https://bz.apache.org/bugzilla/show_bug.cgi?id=63437

Thomas Jarosch <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]
                   |                            |m

--- Comment #2 from Thomas Jarosch <[hidden email]> ---
The log output is created in a custom output filter chained after mod_proxy.
Sorry I didn't give more configuration details, here's the proxy config:

<Proxy *>
   ProxyAddHeaders Off
   SetOutputFilter fsav
</Proxy>


The filter code is like this:

apr_status_t fsav_filter(ap_filter_t *f, apr_bucket_brigade *buckets);

ap_register_output_filter_protocol("fsav", fsav_filter, NULL,
AP_FTYPE_CONTENT_SET, 0);


I've just added this debug logger at the start of the filter function:

    ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, f->r, "[%d] Debug issue #63437:
%s", getpid(), f->r->uri);


Output:
[Wed May 15 17:02:44.468188 2019] [fsav:error] [pid 19206:tid 3062889280]
[client 127.0.0.1:44788] [19206] Debug issue #63437: http:/eicar.org/

-> the URI is already broken.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]



--
Eric Covener
[hidden email]