ekino/EkinoNewRelicBundle — pull request #7 by mlebarron (2012-12-03 22:16:34+00:00)
Title: Issue reporting everything as app.php

We have just been tracking an issue with newrelic where many of our transactions were being tracked against app.php instead of the route name.

Newrelic's advice was to ensure we always call new relic_set_appname() before any other calls.

I switched the order of the setTransactionName and setApplicationName in Ekino\Bundle\NewRelicBundle\Listener\RequestListener::onCoreRequest() and we now have consistent reporting against route names.

i.e.:
$this->interactor->setTransactionName($route ?: 'symfony unknow route');
$this->interactor->setApplicationName($this->newRelic->getName());

becomes:

$this->interactor->setApplicationName($this->newRelic->getName());
$this->interactor->setTransactionName($route ?: 'symfony unknow route');

This commit resolves the issue.