feat: Run nameTransaction on router.matched event so that exceptions receive the correct transaction name (#50)

lukewaite · pleckey · commit f5ee86626313 · 2017-03-28T11:41:05.000-04:00
diff --git a/src/Intouch/LaravelNewrelic/NewrelicServiceProvider.php b/src/Intouch/LaravelNewrelic/NewrelicServiceProvider.php
@@ -74,8 +74,8 @@ protected function registerNamedTransactions()
 	{
 		$me  = $this;
 		$app = $this->app;
-		$app['router']->after(
-			function ( $request, $response ) use ( $me ) {
+		$app['router']->matched(
+			function ( ) use ( $me ) {
 				if (true == $me->app['config']->get( 'newrelic.auto_name_transactions' )) {
 					$me->app['newrelic']->nameTransaction( $me->getTransactionName() );
 				}

Original file line number	Diff line number	Diff line change
`@@ -74,8 +74,8 @@ protected function registerNamedTransactions()`
`74`	`74`	`{`
`75`	`75`	`$me = $this;`
`76`	`76`	`$app = $this->app;`
`77`		`- $app['router']->after(`
`78`		`- function ( $request, $response ) use ( $me ) {`
	`77`	`+ $app['router']->matched(`
	`78`	`+ function ( ) use ( $me ) {`
`79`	`79`	`if (true == $me->app['config']->get( 'newrelic.auto_name_transactions' )) {`
`80`	`80`	`$me->app['newrelic']->nameTransaction( $me->getTransactionName() );`
`81`	`81`	`}`