FS#1691 DokuWiki rewrite doesn't work when doku.php is not under the document root

Backend

• 

  2009-05-12 Michitux

  So far I could see this bug under Solaris using Lighttpd and a user reported exactly the same problem in the IRC channel today, he is using Debian Lenny, Apache 2 and the DokuWiki dpkg: $_SERVER['DOCUMENT_ROOT'] and $_SERVER['SCRIPT_FILENAME'] contain two completely different directories (e.g. the document root is the global document root while the script filename is in the user directory or /var/www/ and /usr/share/dokuwiki/doku.php in the case of the Debian system). Therefore, the following code from inc/pageutils.php that tries to get the script filename completely fails and DokuWiki is unable to extract the page id from the url:
  
     if($conf['basedir']){
       $relpath = '';
       if($param != 'id') {
         $relpath = 'lib/exe/';
       }
       $script = $conf['basedir'].$relpath.basename($_SERVER['SCRIPT_FILENAME']);
     }elseif($_SERVER['DOCUMENT_ROOT'] && $_SERVER['SCRIPT_FILENAME']){
       $script = preg_replace ('/^'.preg_quote($_SERVER['DOCUMENT_ROOT'],'/').'/','',
                               $_SERVER['SCRIPT_FILENAME']);
       $script = '/'.$script;
     }else{
       $script = $_SERVER['SCRIPT_NAME'];
     }
  
  I have been able to fix it by replacing the last two alternatives and instead checking for the presence of $_SERVER['SCRIPT_NAME'], but I have no idea if that will work in all setups or if it will cause the same problem on other systems. Another alternative would be to check if the script filename contains the document root and if not to use the other alternative.
• 

  2009-05-14 andi

  This is one of the functions we have unit tests for. Could you please provide failing unit tests for the conditions? Then your patch can be checked against all tests and applied if it works.
• 

  2009-05-14 andi

  Now that I'm thinking about it. It might be a configuration problem of the lighty/fastcgi setup...
  
  Could you check if setting cgi.fix_pathinfo in php.ini and/or setting the broken-scriptfilename option in the lighty conf fixes the problem?
• 

  2009-05-15 andi

  should be fixed now