From 5fde58d4f0c6bc0dbafaac772d4b0c46fe7e7c9b Mon Sep 17 00:00:00 2001 From: Robin Appelman Date: Thu, 25 May 2023 17:03:18 +0200 Subject: [PATCH] log failures to read certificates during listing Signed-off-by: Robin Appelman --- lib/private/Security/CertificateManager.php | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/lib/private/Security/CertificateManager.php b/lib/private/Security/CertificateManager.php index 6231534205a6..9f824f56bb25 100644 --- a/lib/private/Security/CertificateManager.php +++ b/lib/private/Security/CertificateManager.php @@ -92,8 +92,14 @@ public function listCertificates(): array { while (false !== ($file = readdir($handle))) { if ($file != '.' && $file != '..') { try { - $result[] = new Certificate($this->view->file_get_contents($path . $file), $file); + $content = $this->view->file_get_contents($path . $file); + if ($content !== false) { + $result[] = new Certificate($content, $file); + } else { + $this->logger->error("Failed to read certificate from $path"); + } } catch (\Exception $e) { + $this->logger->error("Failed to read certificate from $path", ['exception' => $e]); } } }