Merge pull request #2418 from nextcloud/fix/filename-requirements

nextcloud · Aug 13, 2024 · c6d57a2 · c6d57a2
2 parents ca91f53 + bf06f3f
commit c6d57a2
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/js/viewer-main.mjs b/js/viewer-main.mjs
@@ -26978,6 +26978,9 @@ const genFileInfo = function(obj) {
 };
 function getDavPath({ filename, source = "" }) {
   const prefixUser = davRootPath;
+  if (!filename || typeof filename !== "string") {
+    return null;
+  }
   if (source && !source.includes(prefixUser)) {
     return null;
   }

diff --git a/js/viewer-main.mjs.map b/js/viewer-main.mjs.map
diff --git a/src/utils/fileUtils.ts b/src/utils/fileUtils.ts
@@ -125,6 +125,10 @@ function getDavPath({ filename, source = '' }: { filename: string, source?: stri
 	// https://github.com/nextcloud/server/issues/19700
 	const prefixUser = davRootPath
 
+	if (!filename || typeof filename !== 'string') {
+		return null
+	}
+
 	// If we have a source but we're not a dav resource, return null
 	if (source && !source.includes(prefixUser)) {
 		return null