Optimierung: Interne Klassen zum Einlesen der Vorschau verwenden

[jan.theofel]

Hallo,

in der Erweiterung wird die Funktion fopen($url) verwendet um (falls aktiviert) die Vorschaubilder der verlinkten Webseiten von einem Webdienst zu laden.

Ich möchte vorschlagen, dass auf die internen Contao-Funktionen umzuschreiben. Also in etwa so:
$objRequest = new Request();
$objRequest->send($url);

Vorteil:
Du nutzt die Contao-eigenen Funktionen, sprich es ist sichergestellt, dass die das funktioniert. Nutzt man hingegen direkt fopen() könnte das für URLs aus Sicherheitsgründen durch den Hoster deaktiviert sein um sich gegen dämlichen Code der Art fopen($_GET['foo']) abzusichern. (Siehe auch hier).

Jan

[psren]

Nutzt man hingegen direkt fopen() könnte das für URLs aus Sicherheitsgründen durch den Hoster deaktiviert sein

Wie es zum Beispiel bei 1&1 der Fall ist.

[jan.theofel]

Hi,

... und auf unseren Servern auch. ;-) Da kann ich das zwar pro vhost wieder aktivieren, aber da ich nicht weiß ob die Kunden neben Contao noch andere Sachen hinladen mache ich das doch nicht so gerne.

Jan

[arnefi]

Hi Jan,

das klingt super. :-) Werde ich in den nächsten Tagen nachbessern.

Vielen Dank!

Beste Grüße,
Arne