Yes, I did ask Stig who's code it is and he said
it was propably left commented by accident.
And I asked about this on php-dev a while ago..without
anybody bothering to reply.
So, what is the deep meaning for that to be different
from what our manual says?
--Jani
On Thu, 13 Jun 2002, Rasmus Lerdorf wrote:
>Uh Jani, how much research did you do on this one? Don't you think there
>is a reason that this code is commented out?
>
>On Thu, 13 Jun 2002, Jani Taskinen wrote:
>
>> sniper Thu Jun 13 18:13:58 2002 EDT
>>
>> Modified files:
>> /php4/sapi/apache mod_php4.c
>> Log:
>> @- Fixed bug with Apache which let PHP_AUTH_* variables to be set when
>> @ external basic auth mechanism was used. (Jani)
>>
>> Fixes bugs: #16653, #14534, #14370
>>
>>
>>
>> Index: php4/sapi/apache/mod_php4.c
>> diff -u php4/sapi/apache/mod_php4.c:1.132 php4/sapi/apache/mod_php4.c:1.133
>> --- php4/sapi/apache/mod_php4.c:1.132 Fri May 31 00:47:15 2002
>> +++ php4/sapi/apache/mod_php4.c Thu Jun 13 18:13:57 2002
>> @@ -17,7 +17,7 @@
>> | PHP 4.0 patches by Zeev Suraski <[EMAIL PROTECTED]> |
>> +----------------------------------------------------------------------+
>> */
>> -/* $Id: mod_php4.c,v 1.132 2002/05/31 04:47:15 rvenkat Exp $ */
>> +/* $Id: mod_php4.c,v 1.133 2002/06/13 22:13:57 sniper Exp $ */
>>
>> #include "php_apache_http.h"
>>
>> @@ -403,7 +403,7 @@
>> authorization = table_get(r->headers_in, "Authorization");
>> }
>> if (authorization
>> -/* && !auth_type(r) */
>> + && !auth_type(r)
>> && !strcasecmp(getword(r->pool, &authorization, ' '), "Basic")) {
>> tmp = uudecode(r->pool, authorization);
>> SG(request_info).auth_user = getword_nulls_nc(r->pool, &tmp, ':');
>>
>>
>>
>> --
>> PHP CVS Mailing List (http://www.php.net/)
>> To unsubscribe, visit: http://www.php.net/unsub.php
>>
>
--
--
PHP Development Mailing List <http://www.php.net/>
To unsubscribe, visit: http://www.php.net/unsub.php