@fredli74 opened this Pull Request on December 6th 2014

Implemented navigator.sendBeacon as alternative tracking method. (#6641)
I choose to disabled beacon support by default as sendBeacon requests are POST requests (cross
domain awareness). It is easily enabled with _paq.push(["enableBeacon”]);.

It uses the same format as bulk requests to avoid posting parameters as forms (resulting in a lot of overhead data).

@mattab commented on December 10th 2014 Member

Thanks for the PR @fredli74
it would be nice to add automated tests for this feature to check in the future that it will always work. you can find the existing tests in: https://github.com/piwik/piwik/blob/master/tests/javascript/index.php#L405

the tests will be executed after each of your commits visible on travis CI in the JavaScriptTest build

@mattab commented on April 7th 2015 Member

Maybe someone from the team would be interested to take a look at @fredli74 PR and maybe add tests or a review?

@mattab commented on July 25th 2016 Member

Thank you for this proposed pull request.

Because it was last updated more than one month ago, it is our policy to close pull requests opened for a long time without updates. If you would like to continue work on the pull request, please simply ping us to have it re-opened (after you have pushed a new commit).

We hope you understand this and we look forward to seeing an update from you on this pull request or another one!


@mattab commented on March 4th 2018 Member

FYI we are implementing support for sendBeacon in https://github.com/matomo-org/matomo/pull/12538

This Pull Request was closed on July 25th 2016
Powered by GitHub Issue Mirror