Only proxy images through cover proxy.

demiankatz · Feb 5, 2024 · 8e33ade · 8e33ade
1 parent 40ae884
commit 8e33ade
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/module/VuFind/src/VuFind/Controller/CoverController.php b/module/VuFind/src/VuFind/Controller/CoverController.php
@@ -129,10 +129,13 @@ public function showAction()
         if (!empty($url)) {
             try {
                 $image = $this->proxy->fetch($url);
-                return $this->displayImage(
-                    $image->getHeaders()->get('content-type')->getFieldValue(),
-                    $image->getContent()
-                );
+                $contentType = $image->getHeaders()->get('content-type')->getFieldValue();
+                if (str_starts_with($contentType, 'image/')) {
+                    return $this->displayImage(
+                        $contentType,
+                        $image->getContent()
+                    );
+                }
             } catch (\Exception $e) {
                 // If an exception occurs, drop through to the standard case
                 // to display an image unavailable graphic.

diff --git a/module/VuFind/tests/.phpunit.cache/test-results b/module/VuFind/tests/.phpunit.cache/test-results