Bug in der test.php

  • Betroffene App
    WoltLab Suite Core

    Hallo,

    Ich habe neulich einen Fehler in der test.php entdeckt,
    Es wird bei lighttpd und anderen Servern eine Warnung im Bezug auf nginx angezeigt.

    Dies kann behoben werden in dem folgende Änderung vorgenommen wird:

    Code
    $isNginx = false;
        if ((isset($_SERVER['SERVER_SOFTWARE'])) && (stripos($_SERVER['SERVER_SOFTWARE'], 'nginx') !== false)) {
            $isNginx = true;
        }
        $isNginx = true;

    Zu:

    Code
    $isNginx = false;
        if ((isset($_SERVER['SERVER_SOFTWARE'])) && (stripos($_SERVER['SERVER_SOFTWARE'], 'nginx') !== false)) {
            $isNginx = true;
        }

    Ich wusste nicht wo so etwas reingehört also habe ich es einfach mal hier reingetan.

    • Offizieller Beitrag

    Das ist kein Fehler, ich hatte lediglich die Prüfung auf PATH_INFO ursprünglich nur bei nginx durchführen lassen. Allerdings gibt es genau so auch Apache-Server die nicht korrekt konfiguriert sind und dementsprechend wird der Test bei jeder Installation durchgeführt. Ich hätte vielleicht eher das ganze Konstrukt anpassen sollen, denn das nachfolgende if ist immer true (und soll es auch sein).

    Lange Rede, kurzer Sinn: Unschön ja, aber definitiv kein Fehler.

    Alexander Ebert
    Senior Developer WoltLab® GmbH

  • Allerdings zeigt es bei Lighttpd auch mit funktionierender PATH_INFO einen Fehler an ...
    der PATH_INFO Test wird als PASSED angezeigt. Es wird trotzdem in rot gewarnt, dass PATH_INFO nicht funktioniert.

Jetzt mitmachen!

Sie haben noch kein Benutzerkonto auf unserer Seite? Registrieren Sie sich kostenlos und nehmen Sie an unserer Community teil!