From 64acead53a49990ba53674686fb984d695ca054c Mon Sep 17 00:00:00 2001 From: Zivko Sudarski <53841402+zivkos@users.noreply.github.com> Date: Fri, 21 Feb 2025 23:22:31 +1300 Subject: [PATCH] Fix Incorrect file name from Download component DownloadServlet.java (#2903) Fixes #2751 --- .../wcm/core/components/internal/servlets/DownloadServlet.java | 2 +- .../core/components/internal/servlets/DownloadServletTest.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/bundles/core/src/main/java/com/adobe/cq/wcm/core/components/internal/servlets/DownloadServlet.java b/bundles/core/src/main/java/com/adobe/cq/wcm/core/components/internal/servlets/DownloadServlet.java index 5f0d5827de..e8cd7cb49f 100644 --- a/bundles/core/src/main/java/com/adobe/cq/wcm/core/components/internal/servlets/DownloadServlet.java +++ b/bundles/core/src/main/java/com/adobe/cq/wcm/core/components/internal/servlets/DownloadServlet.java @@ -191,7 +191,7 @@ private void sendResponse(InputStream stream, long size, String mimeType, String response.setContentLength((int) size); } if (inline) { - response.setHeader(CONTENT_DISPOSITION_HEADER, "inline"); + response.setHeader(CONTENT_DISPOSITION_HEADER, "inline; filename=\"" + filename + "\""); response.setHeader(CSP_HEADER, "sandbox"); } else { response.setHeader(CONTENT_DISPOSITION_HEADER, "attachment; filename=\"" + filename + "\""); diff --git a/bundles/core/src/test/java/com/adobe/cq/wcm/core/components/internal/servlets/DownloadServletTest.java b/bundles/core/src/test/java/com/adobe/cq/wcm/core/components/internal/servlets/DownloadServletTest.java index 91bc5ec47b..8c27ec31d7 100644 --- a/bundles/core/src/test/java/com/adobe/cq/wcm/core/components/internal/servlets/DownloadServletTest.java +++ b/bundles/core/src/test/java/com/adobe/cq/wcm/core/components/internal/servlets/DownloadServletTest.java @@ -85,7 +85,7 @@ void testInlineAssetDownload() throws Exception { context.requestPathInfo().setExtension("pdf"); downloadServlet.doGet(context.request(), context.response()); assertTrue(context.response().containsHeader("Content-Disposition")); - assertEquals("inline", context.response().getHeader("Content-Disposition")); + assertEquals("inline; filename=\"Download_Test_PDF.pdf\"", context.response().getHeader("Content-Disposition")); assertTrue(context.response().containsHeader("Content-Security-Policy")); assertEquals("sandbox", context.response().getHeader("Content-Security-Policy")); assertEquals(8192, context.response().getBufferSize());