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