diff options
| author | Richard van der Hoff <github@rvanderhoff.org.uk> | 2017-11-27 12:07:52 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-11-27 12:07:52 +0000 |
| commit | 96387bd26f39a6ffc0903ee35ae761a3110fa077 (patch) | |
| tree | 11b163a4d5f5ce4d6a0351a1bafdcd1b2ef92558 | |
| parent | Merge pull request #2701 from matrix-org/rav/one_mediarepo_to_rule_them_all (diff) | |
| parent | Improve tracebacks on exceptions (diff) | |
| download | synapse-96387bd26f39a6ffc0903ee35ae761a3110fa077.tar.xz | |
Merge pull request #2705 from matrix-org/rav/improve_tracebacks
Improve tracebacks on exceptions
| -rw-r--r-- | synapse/http/server.py | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/synapse/http/server.py b/synapse/http/server.py index 3ca1c9947c..25466cd292 100644 --- a/synapse/http/server.py +++ b/synapse/http/server.py @@ -28,6 +28,7 @@ from canonicaljson import ( ) from twisted.internet import defer +from twisted.python import failure from twisted.web import server, resource from twisted.web.server import NOT_DONE_YET from twisted.web.util import redirectTo @@ -131,12 +132,17 @@ def wrap_request_handler(request_handler, include_metrics=False): version_string=self.version_string, ) except Exception: - logger.exception( - "Failed handle request %s.%s on %r: %r", + # failure.Failure() fishes the original Failure out + # of our stack, and thus gives us a sensible stack + # trace. + f = failure.Failure() + logger.error( + "Failed handle request %s.%s on %r: %r: %s", request_handler.__module__, request_handler.__name__, self, - request + request, + f.getTraceback().rstrip(), ) respond_with_json( request, |