mirror of
				https://git.anonymousland.org/anonymousland/synapse.git
				synced 2025-11-04 09:55:40 -05:00 
			
		
		
		
	Improve tracebacks on exceptions
Use failure.Failure to recover our failure, which will give us a useful stacktrace, unlike the rethrown exception.
This commit is contained in:
		
							parent
							
								
									79eba878a7
								
							
						
					
					
						commit
						6be01f599b
					
				
					 1 changed files with 9 additions and 3 deletions
				
			
		| 
						 | 
				
			
			@ -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,
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue