log failures to read certificates during listing

Signed-off-by: Robin Appelman <robin@icewind.nl>

diff --git a/lib/private/Security/CertificateManager.php b/lib/private/Security/CertificateManager.php
index 6231534205a6cc38a170e8a7f7681c36b99af0cc..9f824f56bb2566de6d44583338ed4bbdd33fd3c0 100644 (file)
--- a/lib/private/Security/CertificateManager.php
+++ b/lib/private/Security/CertificateManager.php
@@ -92,8 +92,14 @@ class CertificateManager implements ICertificateManager {
                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]);
                                }
                        }
                }