From 9106dd343f8f95e4e40cf0b9d70f9826301e9db2 Mon Sep 17 00:00:00 2001 From: Franco Fichtner Date: Tue, 11 Apr 2017 14:49:56 +0200 Subject: [PATCH] net/acme-client: try to fix "Node no longer exists" As this harmless warning makes up most of our crash reports, it's time to do something against this. It' doesn't look totally right, but following a similar case on github this might do the trick. See: https://github.com/Atlantic18/DoctrineExtensions/pull/1627/files --- .../src/opnsense/scripts/OPNsense/AcmeClient/certhelper.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/security/acme-client/src/opnsense/scripts/OPNsense/AcmeClient/certhelper.php b/security/acme-client/src/opnsense/scripts/OPNsense/AcmeClient/certhelper.php index bd2840b34..efab2e964 100755 --- a/security/acme-client/src/opnsense/scripts/OPNsense/AcmeClient/certhelper.php +++ b/security/acme-client/src/opnsense/scripts/OPNsense/AcmeClient/certhelper.php @@ -119,7 +119,7 @@ function cert_action_validator($opt_cert_id) // Search for cert ID in configuration $configObj = Config::getInstance()->object(); - if (isset($configObj->OPNsense->AcmeClient->certificates)) { + if (isset($configObj->OPNsense->AcmeClient->certificates) && $configObj->OPNsense->AcmeClient->certificates->count() > 0) { foreach ($configObj->OPNsense->AcmeClient->certificates->children() as $certObj) { // Extract cert ID $cert_id = (string)$certObj->id;