For better testability and so we don't have to use HTTP requests to test the tracker, exit & die should not be called from within core/Tracker.php. These calls should be moved to users of core/Tracker.php, which I believe is just core/piwik.php.
FYI: Without having a look we might still have to call exit in case of a redirect or at least make sure the script ends after setting the location header. Otherwise :+1:
@tsteur I think in such case we can throw an exception and catch in piwik.php?
Not sure if such a good idea. Just had a look it uses
Url::redirectToUrl() anyway so there is no need to exit in Tracker.php anyway.
Yes, there should be no exit anymore apart from
redirectToUrl which triggers an exception on cli (phpunit)