fix: support poorly formatted graphql queries fixes #385

Author: oliemansm

graphql-java-servlet/src/main/java/graphql/kickstart/servlet/GraphQLPostInvocationInputParser.java

@@ -10,6 +10,7 @@
 import graphql.kickstart.servlet.input.GraphQLInvocationInputFactory;
 import java.io.IOException;
 import java.util.List;
+import java.util.stream.Collectors;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
@@ -27,12 +28,12 @@ class GraphQLPostInvocationInputParser extends AbstractGraphQLInvocationInputPar
   public GraphQLInvocationInput getGraphQLInvocationInput(
       HttpServletRequest request, HttpServletResponse response) throws IOException {
     if (APPLICATION_GRAPHQL.equals(request.getContentType())) {
-      String query = request.getReader().lines().collect(joining());
+      String query = request.getReader().lines().collect(joining(" "));
       GraphQLRequest graphqlRequest = GraphQLRequest.createQueryOnlyRequest(query);
       return invocationInputFactory.create(graphqlRequest, request, response);
     }
 
-    String body = request.getReader().lines().collect(joining());
+    String body = request.getReader().lines().collect(joining(" "));
     if (isSingleQuery(body)) {
       GraphQLRequest graphqlRequest = graphQLObjectMapper.readGraphQLRequest(body);
       return invocationInputFactory.create(graphqlRequest, request, response);

graphql-java-servlet/src/test/groovy/graphql/kickstart/servlet/AbstractGraphQLHttpServletSpec.groovy

@@ -422,6 +422,25 @@ class AbstractGraphQLHttpServletSpec extends Specification {
     getResponseContent().data.echo == "test"
   }
 
+  def "query over HTTP POST multiline body returns data"() {
+    setup:
+    request.setContent("""
+        query { object {
+a
+b
+        } }""".bytes)
+    request.setMethod("POST")
+    request.contentType = "application/graphql"
+
+    when:
+    servlet.doPost(request, response)
+
+    then:
+    response.getStatus() == STATUS_OK
+    response.getContentType() == CONTENT_TYPE_JSON_UTF8
+    getResponseContent().data.object.b == null
+  }
+
   def "disabling async support on request over HTTP POST does not start async request"() {
     setup:
     servlet = TestUtils.createDefaultServlet({ env -> env.arguments.arg }, { env -> env.arguments.arg }, { env ->