Validate the file extensions using lower case

Previously a check could fail if, for example, “.png” is allowed but the uploaded file uses the uppercase “.PNG“ extension.

diff --git a/ts/WoltLabSuite/Core/Component/File/Upload.ts b/ts/WoltLabSuite/Core/Component/File/Upload.ts
index a44178222b1087c49b20df62f3b41cde11df0149..8f7fe935e43818cb35d253086a4cd84d35710ae5 100644 (file)
--- a/ts/WoltLabSuite/Core/Component/File/Upload.ts
+++ b/ts/WoltLabSuite/Core/Component/File/Upload.ts
@@ -239,11 +239,11 @@ function validateFileSize(element: WoltlabCoreFileUploadElement, file: File): bo
 }
 
 function validateFileExtension(element: WoltlabCoreFileUploadElement, file: File): boolean {
-  const fileExtensions = (element.dataset.fileExtensions || "*").split(",");
+  const fileExtensions = (element.dataset.fileExtensions || "*").toLowerCase().split(",");
   for (const fileExtension of fileExtensions) {
     if (fileExtension === "*") {
       return true;
-    } else if (file.name.endsWith(fileExtension)) {
+    } else if (file.name.toLowerCase().endsWith(fileExtension)) {
       return true;
     }
   }

diff --git a/wcfsetup/install/files/js/WoltLabSuite/Core/Component/File/Upload.js b/wcfsetup/install/files/js/WoltLabSuite/Core/Component/File/Upload.js
index 157795b9358eb43d7a8de530a0bb2d2925a76209..227c8713445238973f8a15de24d7aa73d95e4b25 100644 (file)
--- a/wcfsetup/install/files/js/WoltLabSuite/Core/Component/File/Upload.js
+++ b/wcfsetup/install/files/js/WoltLabSuite/Core/Component/File/Upload.js
@@ -157,12 +157,12 @@ define(["require", "exports", "tslib", "WoltLabSuite/Core/Helper/Selector", "Wol
         return false;
     }
     function validateFileExtension(element, file) {
-        const fileExtensions = (element.dataset.fileExtensions || "*").split(",");
+        const fileExtensions = (element.dataset.fileExtensions || "*").toLowerCase().split(",");
         for (const fileExtension of fileExtensions) {
             if (fileExtension === "*") {
                 return true;
             }
-            else if (file.name.endsWith(fileExtension)) {
+            else if (file.name.toLowerCase().endsWith(fileExtension)) {
                 return true;
             }
         }