diff options
| author | Yann Ylavic <ylavic@apache.org> | 2024-08-01 16:43:58 +0200 |
|---|---|---|
| committer | Yann Ylavic <ylavic@apache.org> | 2024-08-01 16:43:58 +0200 |
| commit | 13963ef367a97bb51836121b5e53a739df7e92c1 (patch) | |
| tree | 73c4e7250816bd9dd1e5002e4c8728a833653184 /changes-entries | |
| parent | Follow up to r1919617: Better CHANGES entry per Eric. (diff) | |
| download | apache2-13963ef367a97bb51836121b5e53a739df7e92c1.tar.xz apache2-13963ef367a97bb51836121b5e53a739df7e92c1.zip | |
mod_proxy_fcgi: Don't re-encode SCRIPT_FILENAME. PR 69203
Before r1918550 (r1918559 in 2.4.60), "SetHandler proxy:..." configurations
did not pass through proxy_fixup() hence the proxy_canon_handler hooks, leaving
fcgi's SCRIPT_FILENAME environment variable (from r->filename) decoded, or more
exactly not re-encoded.
We still want to call ap_proxy_canon_url() for "fcgi:" to handle/strip the UDS
"unix:" case and check that r->filename is valid and contains no controls, but
proxy_fcgi_canon() will not ap_proxy_canonenc_ex() thus re-encode anymore.
Note that this will do the same for "ProxyPass fcgi:...", there is no reason
that using SetHandler or ProxyPass don't result in the same thing. If an opt
in/out makes sense we should probably look at ProxyFCGIBackendType.
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1919620 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'changes-entries')
| -rw-r--r-- | changes-entries/bz69203.txt | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/changes-entries/bz69203.txt b/changes-entries/bz69203.txt new file mode 100644 index 0000000000..3613dad763 --- /dev/null +++ b/changes-entries/bz69203.txt @@ -0,0 +1 @@ +mod_proxy_fcgi: Don't re-encode SCRIPT_FILENAME. PR 69203. [Yann Ylavic]
\ No newline at end of file |