Skip to content

Commit dfe7f55

Browse files
authored
Merge pull request #219 from apae89/feature/fix_content_length
Fix: Erroneous calculation of content-length can lead to errors on …
2 parents fe62e32 + d65b1d9 commit dfe7f55

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

graphql-java-servlet/src/main/java/graphql/servlet/SingleQueryResponseWriter.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@
66
import graphql.ExecutionResult;
77
import graphql.kickstart.execution.GraphQLObjectMapper;
88
import java.io.IOException;
9+
import java.nio.charset.Charset;
10+
import java.nio.charset.StandardCharsets;
911
import javax.servlet.http.HttpServletRequest;
1012
import javax.servlet.http.HttpServletResponse;
1113
import lombok.RequiredArgsConstructor;
@@ -21,7 +23,7 @@ public void write(HttpServletRequest request, HttpServletResponse response) thro
2123
response.setContentType(APPLICATION_JSON_UTF8);
2224
response.setStatus(STATUS_OK);
2325
String responseContent = graphQLObjectMapper.serializeResultAsJson(result);
24-
response.setContentLength(responseContent.length());
26+
response.setContentLength(responseContent.getBytes(StandardCharsets.UTF_8).length);
2527
response.getWriter().write(responseContent);
2628
}
2729

0 commit comments

Comments
 (0)