Filename conflict

I've just tested this patch and noticed I can continuously append .avif on to any image path, e.g image.jpg.avif, image.jpg.avif.avif, image.jpg.avif.avif.avif.

+++ b/src/Controller/ImageStyleDownloadController.php
@@ -113,39 +113,12 @@ class ImageStyleDownloadController extends FileDownloadController {
-    if (!file_exists($image_uri)) {
+    $image_uri = str_replace('.avif', '', $image_uri);
+    if ($avifWanted && !file_exists($image_uri)) {

We need to only strip the last .avif from the uri rather than replacing all instances of .avif

Attached patch addresses #7 and #8.

I have opened an MR that incorporates #9 as well as ✨ Do not regenerate the avif file each time it is requested Needs work and 📌 Test coverage Needs review .

Nice change!

Automatically closed - issue fixed for 2 weeks with no activity.