Merge branch '5.8.x' into 6.0.x

spring-projects · Sep 25, 2023 · 1a0b8b7 · 1a0b8b7
2 parents f06c2b9 + 664ee9a
commit 1a0b8b7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/...ain/java/org/springframework/security/saml2/provider/service/web/Saml2MetadataFilter.java b/...ain/java/org/springframework/security/saml2/provider/service/web/Saml2MetadataFilter.java
@@ -88,7 +88,7 @@ private void writeMetadataToResponse(HttpServletResponse response, String regist
 		String encodedFileName = URLEncoder.encode(fileName, StandardCharsets.UTF_8.name());
 		String format = "attachment; filename=\"%s\"; filename*=UTF-8''%s";
 		response.setHeader(HttpHeaders.CONTENT_DISPOSITION, String.format(format, fileName, encodedFileName));
-		response.setContentLength(metadata.length());
+		response.setContentLength(metadata.getBytes(StandardCharsets.UTF_8).length);
 		response.setCharacterEncoding(StandardCharsets.UTF_8.name());
 		response.getWriter().write(metadata);
 	}

diff --git a/...ava/org/springframework/security/saml2/provider/service/web/Saml2MetadataFilterTests.java b/...ava/org/springframework/security/saml2/provider/service/web/Saml2MetadataFilterTests.java
@@ -165,6 +165,8 @@ public void doFilterWhenCharacterEncodingThenEncodeSpecialCharactersCorrectly()
 		this.filter.doFilter(this.request, this.response, this.chain);
 		assertThat(this.response.getCharacterEncoding()).isEqualTo(StandardCharsets.UTF_8.name());
 		assertThat(this.response.getContentAsString(StandardCharsets.UTF_8)).isEqualTo(generatedMetadata);
+		assertThat(this.response.getContentLength())
+				.isEqualTo(generatedMetadata.getBytes(StandardCharsets.UTF_8).length);
 	}
 
 	@Test