Modify

Opened 11 years ago

Closed 11 years ago

#429 closed Bug/Fehler (fixed)

Directory browsing / Admin

Reported by: Mediagenie System Owned by: somebody
Priority: normal Milestone: modified-shop-2.0.0.0
Component: Admin Version: 2.0.0.0
Keywords: Cc:
Blocked By: Blocking:

Description

Hallo,

bei der Version 2.0 habe ich noch folgendes entdeckt:
http://demo.modified-shop.org/tags/modified-2.00/admin/includes/
Verzeichnis ist durchsichtbar. Sicherheitstechnisch wäre es nicht schön.

Ich weiß, dass in der _.htaccess Datei folgendes steht:
# Disable directory browsing (disabled by default as this causes problems on some servers)
#Options All -Indexes

Als Idee könnte man eine index.html oder index.php in alle Verzeichnisse reinlegen, damit directory browsing erstmal deaktiviert wird.

Mit freundlichen Grüßen
Jürgen H.
Mediagenie System UG

Attachments (0)

Change History (8)

comment:1 by Mediagenie System, 11 years ago

Nachtrag:
Bitte noch Fehlertext ausbessern: (Kodierungsproblem)
Fehler 403: Der Server verweigert die Ausf�hrung.

comment:2 by Mediagenie System, 11 years ago

view-source:http://demo.modified-shop.org/tags/modified-2.00/admin/includes/application_top.php

<br />
<b>Fatal error</b>: require_once(): Failed opening required 'includes/functions/compatibility.php' (include_path='.:/usr/share/php:/usr/share/pear') in <b>/var//tags/modified-2.00/admin/includes/application_top.php</b> on line <b>86</b><br />

comment:3 by Gerhard Waldemair, 11 years ago

Resolution: fixed
Status: newclosed

in reply to:  2 ; comment:4 by Torsten Riemer, 11 years ago

Resolution: fixed
Status: closedreopened

Replying to Mediagenie System:

view-source:http://demo.modified-shop.org/tags/modified-2.00/admin/includes/application_top.php

<br />
<b>Fatal error</b>: require_once(): Failed opening required 'includes/functions/compatibility.php' (include_path='.:/usr/share/php:/usr/share/pear') in <b>/var//tags/modified-2.00/admin/includes/application_top.php</b> on line <b>86</b><br />

Kannst du das bitte nochmal genauer erklären, da wir das nicht nachvollziehen können.

Das Problem beim "directory listing" ist, dass der Apache die Direktive "AllowOverride Options" gesetzt haben muss, damit das funktioniert. Es gibt noch Hoster, die das nicht erlauben und bei denen würde der Admin dann überhaupt nicht mehr aufgerufen werde können.
Das ist auch der Grund, weshalb wir im Frontend in der ".htaccess" die Zeile zwar eingefügt, aber entsprechend auskommentiert haben:

# Disable directory browsing (disabled by default as this causes problems on some servers)
#Options All -Indexes
Version 0, edited 11 years ago by Torsten Riemer (next)

in reply to:  4 comment:5 by Mediagenie System, 11 years ago

Datei: admin/includes/application_top.php
Diese Datei hat kein "Direct Access" Schutz. Beim direkten Zugriff bekommt man "Fatal error" Fehlermeldung. Damit es nicht passiert, kann ich eine schnelle Lösung vorschlagen:
Zeile 48-53 zu veändern

// configuration parameters
if (file_exists('includes/local/configure.php')) {
  include('includes/local/configure.php');
} elseif(file_exists('includes/configure.php')) {
  require('includes/configure.php');
} else {
  die( 'Direct Access to this location is not allowed.' );
}

in reply to:  4 comment:6 by Mediagenie System, 11 years ago

Jo, deswegen habe ich meine "Idee"-Vorschlag gepostet.

Replying to Tomcraft:

Das Problem beim "directory listing" ist, dass der Apache die Direktive "AllowOverride Options" gesetzt haben muss, damit das funktioniert. Es gibt noch Hoster, die das nicht erlauben und bei denen würde der Admin dann überhaupt nicht mehr aufgerufen werde können.
Das ist auch der Grund, weshalb wir im Frontend in der ".htaccess" die Zeile zwar eingefügt, aber entsprechend auskommentiert haben:

comment:7 by Volker Strähle, 11 years ago

Wäre etwas wie

if ($PHP_SELF == DIR_WS_ADMIN . "application_top.php"){ die("Du kommst nicht rein"); }

besser als auf eine vorhendene Datei zu prüfen?

Sollte eine Datei admin/includes/includes/configure.php existieren würde man in o. g. Fall durch kommen (und wieder ind er Zeile "require_once (DIR_WS_INCLUDES.'compatibility.php');" scheitern).

Mit require_once (DIR_FS_ADMIN . DIR_WS_INCLUDES.'compatibility.php'); käme der Fehler nicht, macht aber keine Sinn.

comment:8 by Gerhard Waldemair, 11 years ago

Resolution: fixed
Status: reopenedclosed

mit einer htaccess im includes Ordner ist alles gut.

Modify Ticket

Action
as closed The owner will remain somebody.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.