It's ok to encode the file name in Content-Disposition header, but it's not neccesary to encode blanks. (#799)

Issue: 105672

Author: iroqueta

java/src/main/java/com/genexus/webpanels/WebUtils.java

@@ -329,7 +329,7 @@ public static String getEncodedContentDisposition(String value, int browserType)
 		}
 
 		String filename = value.substring(eqIdx + 1).trim();
-		value = value.substring(0, filenameIdx) + String.format("filename*=UTF-8''%1$s; filename=\"%1$s\"", PrivateUtilities.URLEncode(filename, "UTF8"));
+		value = value.substring(0, filenameIdx) + String.format("filename*=UTF-8''%1$s; filename=\"%1$s\"", PrivateUtilities.URLEncode(filename, "UTF8").replace('+', ' '));
 
 		return value;
 	}

java/src/test/java/com/genexus/webpanels/TestWebUtils.java

@@ -55,6 +55,12 @@ public void TestContentDispositionHeaderEncoding6() {
 		doTest(contentDisposition, expectedContentDisposition);
 	}
 
+	@Test
+	public void TestContentDispositionHeaderEncoding7() {
+		String contentDisposition = "attachment; filename=My file.pdf";
+		String expectedContentDisposition = "attachment; filename*=UTF-8''My file.pdf; filename=\"My file.pdf\"";
+		doTest(contentDisposition, expectedContentDisposition);
+	}
 
 	private void doTest(String contentDisposition, String expectedContentDisposition) {
 		doTest(contentDisposition, expectedContentDisposition, HttpContextWeb.BROWSER_CHROME);