fix(utils): simplified 'getFileExtension()' to pass all test cases

videojs · Feb 19, 2025 · 5e703b6 · 5e703b6
1 parent a14e493
commit 5e703b6
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/src/js/utils/url.js b/src/js/utils/url.js
@@ -47,14 +47,12 @@ export const getAbsoluteURL = function(url) {
  */
 export const getFileExtension = function(path) {
   if (typeof path === 'string') {
-    const splitPathRe = /^(\/?)([\s\S]*?)((?:\.{1,2}|[^\/]+?)(\.*([^\.\/\?]+)))(?:[\/]*|[\?].*)$/;
-    const pathParts = splitPathRe.exec(path);
+    const cleanPath = path.split('?')[0];
 
-    if (pathParts) {
-      return pathParts.pop().toLowerCase();
-    }
-  }
+    const match = cleanPath.match(/\.([^.\/]+)$/);
 
+    return match ? match[1].toLowerCase() : '';
+  }
   return '';
 };