Merge pull request #763 from sangria-graphql/foreach_for_sideeffect

use 'foreach' for side-effects
sangria-graphql · Oct 26, 2021 · 4edbdf2 · 4edbdf2
2 parents 443e7a5 + f6aab7d
commit 4edbdf2
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/modules/core/src/main/scala/sangria/execution/Resolver.scala b/modules/core/src/main/scala/sangria/execution/Resolver.scala
@@ -788,7 +788,7 @@ class Resolver[Ctx](
       newUc.map(_.ctxFn(v)).getOrElse(userCtx)
 
     def resolveError(newUc: Option[MappedCtxUpdate[Ctx, Any, Any]], e: Throwable) = {
-      try newUc.map(_.onError(e))
+      try newUc.foreach(_.onError(e))
       catch {
         case NonFatal(ee) => ee.printStackTrace()
       }