Actually fix the query and caching

src/ImageResizer.php

@@ -25,7 +25,7 @@ try {
     header("Cache-Control: max-age=86400, public");
     echo $image;
 } catch (\Exception $e) {
-    $logger->error($e->getMessage());
+    $logger->error($e->getMessage(), ['exception' => $e, 'path' => $imgPath, 'params' => $params]);
     http_response_code(500);
     echo "Internal Server Error";
 }

src/Lib/SqliteImageDb.php

@@ -27,21 +27,21 @@ class SqliteImageDb implements ImageDb {
     }
 
     public function findImage(string $path, ResizeParams $params): Image | null {
-        $query = "SELECT * FROM images WHERE path = :path";
+        $query = 'SELECT * FROM images WHERE path = :path';
-        $query .= $params->width ? " AND width = :width" : " AND width IS NULL";
+        $query .= $params->width ? ' AND width = :width' : ' AND width IS NULL';
-        $query .= $params->height ? " AND height = :height" : " AND height IS NULL";
+        $query .= $params->height ? ' AND height = :height' : ' AND height IS NULL';
-        $query .= $params->quality ? " AND quality = :quality" : " AND quality IS NULL";
+        $query .= $params->quality ? ' AND quality = :quality' : ' AND quality IS NULL';
 
         $stmt = $this->db->prepare($query);
-        $stmt->execute([
+        $stmt->bindParam(':path', $path);
-            ':path' => $path,
+        if ($params->width !== null) $stmt->bindParam(':width', $params->width);
-            ':width' => $params->width,
+        if ($params->height !== null) $stmt->bindParam(':height', $params->height);
-            ':height' => $params->height,
+        if ($params->quality !== null) $stmt->bindParam(':quality', $params->quality);
-            ':quality' => $params->quality
-        ]);
 
         $this->logger->info("Querying images", ['query' => $stmt->queryString]);
 
+        $stmt->execute();
+
         $result = $stmt->fetchObject();
 
         return $result ? Image::fromString(base64_decode($result->content)) : null;