don't specify environ as argument of the request_started signal (#233)

beniwohli · beniwohli · commit 776a6a98ccc9 · 2018-06-13T22:01:05.000+02:00
Django Channels doesn't provide environ when calling signal handlers. As we don't need it anyway, we can simply accept any arguments. Fixes #232 Closes #233
diff --git a/elasticapm/contrib/django/apps.py b/elasticapm/contrib/django/apps.py
@@ -38,7 +38,7 @@ def register_handlers(client):
 
     request_started.disconnect(dispatch_uid=REQUEST_START_DISPATCH_UID)
     request_started.connect(
-        lambda sender, environ, **kwargs: client.begin_transaction('request')
+        lambda sender, *args, **kwargs: client.begin_transaction('request')
         if _should_start_transaction(client) else None,
         dispatch_uid=REQUEST_START_DISPATCH_UID, weak=False
     )

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ def register_handlers(client):`
`38`	`38`
`39`	`39`	`request_started.disconnect(dispatch_uid=REQUEST_START_DISPATCH_UID)`
`40`	`40`	`request_started.connect(`
`41`		`- lambda sender, environ, **kwargs: client.begin_transaction('request')`
	`41`	`+ lambda sender, args, *kwargs: client.begin_transaction('request')`
`42`	`42`	`if _should_start_transaction(client) else None,`
`43`	`43`	`dispatch_uid=REQUEST_START_DISPATCH_UID, weak=False`
`44`	`44`	`)`