New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
do not call exit/die in core/Tracker.php #6489
Comments
👍 |
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 👍 |
@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 |
This will likely be fixed as part of refactoring in #6075 |
Yes, there should be no exit anymore apart from |
👍 |
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.
The text was updated successfully, but these errors were encountered: