Jump to content
Froxlor Forum
  • 0

PHP Notice bei Cronjob Ausfürung


BNO

Question

Hallo zusammen,

seit heute Morgen spamt mich mein Server mit dieser Fehlermeldung zu:

PHP Notice:  Trying to access array offset on value of type bool in /var/www/froxlor/lib/Froxlor/Cron/Http/LetsEncrypt/AcmeSh.php on line 505

Froxlor Version : 0.10.18
OS: Debian 9 mit Kernel 4.9.0-8-amd64
PHP: 7.4.9

Ich habe dann eben auf 0.10.20 geupdated, aber leider keine Änderung des Verhaltens. Jemand eine Idee was ich da machen kann?

Besten Dank & Gruß

Link to comment
Share on other sites

3 answers to this question

Recommended Posts

  • 1

Also laut code wird vorher schon auf Existenz und Lesbarkeit der Zertifikatsdatei geprüft, so dass dort maximal durch fehlerhaften Inhalt der Dateien die Funktion openssl_x509_parse() fehltschlägt, sonst sollte und dürfte an dieser Stelle dieses Notice gar nicht auftreten.

Verhindern kannst du es mit Hilfe folgender Änderung:

diff --git a/lib/Froxlor/Cron/Http/LetsEncrypt/AcmeSh.php b/lib/Froxlor/Cron/Http/LetsEncrypt/AcmeSh.php
index 3e7338e3..b476f755 100644
--- a/lib/Froxlor/Cron/Http/LetsEncrypt/AcmeSh.php
+++ b/lib/Froxlor/Cron/Http/LetsEncrypt/AcmeSh.php
@@ -502,7 +502,7 @@ class AcmeSh extends \Froxlor\Cron\FroxlorCron
 
                if (is_dir($certificate_folder) && file_exists($ssl_file) && is_readable($ssl_file)) {
                        $cert_data = openssl_x509_parse(file_get_contents($ssl_file));
-                       if ($cert_data['validTo_time_t'] > strtotime($cert_date)) {
+                       if ($cert_data && $cert_data['validTo_time_t'] > strtotime($cert_date)) {
                                return true;
                        }
                }

 

Link to comment
Share on other sites

  • 0

Vielen Dank für die schnelle Antwort!

Ich musste PHP updaten auf eine neure Version als die, die mit Debian 9 ausgeliefert wird, daher hab ich direkt auf 7.4 aktualisiert.

Das Fix scheint auf jeden Fall zu funktioniren. Vielen Dank!

Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now


×
×
  • Create New...