Skip to content
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

Jump to bottom

JS Error tracking should record the same error only once per page view #17163

Closed
tsteur opened this issue Jan 29, 2021 · 0 comments · Fixed by #17288
Closed

JS Error tracking should record the same error only once per page view #17163

tsteur opened this issue Jan 29, 2021 · 0 comments · Fixed by #17288
Assignees
@flamisz
Labels
c: Performance For when we could improve the performance / speed of Matomo. Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement.
Milestone
4.5.0

Comments

@tsteur
Copy link
Member

tsteur commented Jan 29, 2021

When using the enableJSErrorTracking, then we'd want the same error to be reported only once per page view.

Otherwise if there's a loop causing errors, then you could be potentially dealing with thousands of tracking requests per user concurrently.
@tsteur tsteur added c: Performance For when we could improve the performance / speed of Matomo. Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement. labels Jan 29, 2021
@tsteur tsteur added this to the 4.4.0 milestone Jan 29, 2021
@flamisz flamisz self-assigned this Mar 1, 2021
@flamisz flamisz mentioned this issue Mar 2, 2021
Merged
9 tasks
@mattab mattab modified the milestones: 4.7.0, 4.5.0 Aug 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@flamisz flamisz

Labels
c: Performance For when we could improve the performance / speed of Matomo. Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement.
Projects
None yet
Milestone
4.5.0
Development

Successfully merging a pull request may close this issue.

Track JS errors only once per page view matomo-org/matomo
3 participants
@tsteur @mattab @flamisz