From fedaec6c79ebec062e1c2175e54e385b2aa0e6a9 Mon Sep 17 00:00:00 2001
From: Panagiotis Papadopoulos <pano_90@gmx.net>
Date: Fri, 31 Jan 2025 08:33:59 +0100
Subject: [PATCH] refactor(server/utils): merge sanitizeFilenameForHeader into
 getContentDisposition

sanitizeFilenameForHeader is not used anywhere else and is tiny, so let's merge it
---
 src/services/utils.ts | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/src/services/utils.ts b/src/services/utils.ts
index 9af0c20fc..174929334 100644
--- a/src/services/utils.ts
+++ b/src/services/utils.ts
@@ -117,15 +117,10 @@ export async function crash() {
     }
 }
 
-export function sanitizeFilenameForHeader(filename: string) {
-    const sanitizedFilename = sanitize(filename).trim() || "file";
-    return encodeURIComponent(sanitizedFilename);
-}
-
 export function getContentDisposition(filename: string) {
-    const sanitizedFilename = sanitizeFilenameForHeader(filename);
-
-    return `file; filename="${sanitizedFilename}"; filename*=UTF-8''${sanitizedFilename}`;
+    const sanitizedFilename = sanitize(filename).trim() || "file";
+    const uriEncodedFilename = encodeURIComponent(sanitizedFilename);
+    return `file; filename="${uriEncodedFilename}"; filename*=UTF-8''${uriEncodedFilename}`;
 }
 
 // render and book are string note in the sense that they are expected to contain empty string
@@ -325,7 +320,6 @@ export default {
     removeDiacritic,
     removeTextFileExtension,
     replaceAll,
-    sanitizeFilenameForHeader,
     sanitizeSqlIdentifier,
     stripTags,
     timeLimit,