Skip to content

Commit 6a55133

Browse files
authored
Merge pull request #154 from CaselIT/logging
Replaced sys.excepthook with logging.error
2 parents 6df8a63 + a3ba849 commit 6a55133

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

graphql/execution/base.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# -*- coding: utf-8 -*-
2-
import sys
2+
import logging
3+
from traceback import format_exception
34

45
from ..error import GraphQLError
56
from ..language import ast
@@ -11,6 +12,8 @@
1112
from ..utils.type_from_ast import type_from_ast
1213
from .values import get_argument_values, get_variable_values
1314

15+
logger = logging.getLogger(__name__)
16+
1417

1518
class ExecutionContext(object):
1619
"""Data that must be available at all points during query execution.
@@ -87,8 +90,8 @@ def get_argument_values(self, field_def, field_ast):
8790
return result
8891

8992
def report_error(self, error, traceback=None):
90-
sys.excepthook(type(error), error, getattr(
91-
error, 'stack', None) or traceback)
93+
exception = format_exception(type(error), error, getattr(error, 'stack', None) or traceback)
94+
logger.error(''.join(exception))
9295
self.errors.append(error)
9396

9497
def get_sub_fields(self, return_type, field_asts):

0 commit comments

Comments
 (0)