Omit query analysis error message (#282)

* FCORE-562 Omitted QueryAnalysisError message due to security * Returning uuid in error message and logging the error with the same
coursera · Dec 17, 2020 · 21eb5be · 21eb5be
1 parent f7592e3
commit 21eb5be
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/...aphql/src/main/scala/org/coursera/naptime/ari/graphql/controllers/GraphQLController.scala b/...aphql/src/main/scala/org/coursera/naptime/ari/graphql/controllers/GraphQLController.scala
@@ -16,6 +16,7 @@
 
 package org.coursera.naptime.ari.graphql.controllers
 
+import java.util.UUID.randomUUID
 import javax.inject._
 import com.typesafe.scalalogging.Logger
 import com.typesafe.scalalogging.StrictLogging
@@ -147,7 +148,9 @@ class GraphQLController @Inject()(
               }
           }.recover {
             case error: QueryAnalysisError =>
-              OutgoingQuery(error.resolveError.as[JsObject], None)
+              val uuid = randomUUID().toString
+              logger.error("QueryAnalysisError- " + uuid, error)
+              OutgoingQuery(Json.obj("error" -> "QueryAnalysisError", "message" -> uuid), None)
             case error: ErrorWithResolver =>
               OutgoingQuery(error.resolveError.as[JsObject], None)
             case error: Exception =>

diff --git a/version.sbt b/version.sbt
@@ -1 +1 @@
-version in ThisBuild := "0.11.4"
+version in ThisBuild := "0.11.5"