Merge pull request #275 from alexandreBaronZnk/bugfix/250-httprequesthandlerimpl-doesnt-reponse-error-with-detail

Refactor log messages into HttpRequestHandlerImpl #250

Author: oliemansm

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

@@ -37,28 +37,22 @@ public void handle(HttpServletRequest request, HttpServletResponse response) thr
       execute(invocationInput, request, response);
     } catch (GraphQLException e) {
       response.setStatus(STATUS_BAD_REQUEST);
-      log.info("Bad request: cannot create invocation input parser", e);
+      log.info("Bad request: cannot handle http request", e);
       throw e;
     } catch (Throwable t) {
       response.setStatus(500);
-      log.info("Bad request: cannot create invocation input parser", t);
+      log.error("Cannot handle http request", t);
       throw t;
     }
   }
 
   private void execute(GraphQLInvocationInput invocationInput, HttpServletRequest request,
-      HttpServletResponse response) {
-    try {
+      HttpServletResponse response) throws IOException {
       GraphQLQueryResult queryResult = invoke(invocationInput, request, response);
 
       QueryResponseWriter queryResponseWriter = QueryResponseWriter.createWriter(queryResult, configuration.getObjectMapper(),
           configuration.getSubscriptionTimeout());
       queryResponseWriter.write(request, response);
-    } catch (Throwable t) {
-      response.setStatus(STATUS_BAD_REQUEST);
-      log.info("Bad GET request: path was not \"/schema.json\" or no query variable named \"query\" given");
-      log.debug("Possibly due to exception: ", t);
-    }
   }
 
   private GraphQLQueryResult invoke(GraphQLInvocationInput invocationInput, HttpServletRequest request,