Do not call onError in context errors

Urigo · Jul 2, 2019 · 44f686e · 44f686e
1 parent 46b40ca
commit 44f686e
Showing 1 changed file with 0 additions and 15 deletions.
diff --git a/packages/core/src/graphql-module.ts b/packages/core/src/graphql-module.ts
@@ -1020,11 +1020,6 @@ export class GraphQLModule<
                 try {
                   moduleContext = await this.context(info.session, true);
                 } catch (e) {
-                  moduleContext.injector.callHookWithArgsAsync({
-                    hook: 'onError',
-                    args: [e],
-                    instantiate: true
-                  });
                   const logger = this.selfLogger;
                   if ('clientError' in logger) {
                     logger.clientError(e);
@@ -1036,11 +1031,6 @@ export class GraphQLModule<
                 try {
                   result = await resolver.call(typeResolvers, root, moduleContext, info);
                 } catch (e) {
-                  moduleContext.injector.callHookWithArgsAsync({
-                    hook: 'onError',
-                    args: [e],
-                    instantiate: true
-                  });
                   throw e;
                 }
                 return result;
@@ -1057,11 +1047,6 @@ export class GraphQLModule<
                 try {
                   moduleContext = await this.context(info.session, true);
                 } catch (e) {
-                  moduleContext.injector.callHookWithArgsAsync({
-                    hook: 'onError',
-                    args: [e],
-                    instantiate: true
-                  });
                   const logger = this.selfLogger;
                   if ('clientError' in logger) {
                     logger.clientError(e);