Re: cvs: php4 /sapi/apache2filter sapi_apache2.c /sapi/apache2handlersapi_apache2.c

Moriyoshi Koizumi wrote:

Shane Caraveo <shane@caraveo.com> wrote:

This is incorrect behaviour for PATH_TRANSLATED. PATH_TRANSLATED is the translated path of PATH_INFO, *not* of the script being executed. See http://cgi-spec.golux.com/ Shane

I know that. PATH_TRANSLATED is set to the SCRIPT_FILENAME only in case it is not populated, it's just the same way as the apache1 SAPI. Is anything going wrong with this? Moriyoshi

It is technicaly wrong to do that. PATH_TRANSLATED should be empty if no PATH_INFO is available. People are using PATH_TRANSLATED in places where they should be using SCRIPT_FILENAME, and thus the reason for the bug report in the first place. Since the apache 2 module is new, it should be done right, and for PHP5, I think all the sapi modules should do this right. For PHP5, cgi, isapi and nsapi (I beleive I saw the fix for nsapi a couple months back) sapi modules are all handling this per spec. Shane