Skip to content

Breadcrumb added for every mousemove event #724

Closed
@pkaminski

Description

@pkaminski

There's a bug where events other than click and keypress will automatically be tracked as breadcrumbs; this is particularly annoying for mousemove events since they'll fill up the breadcrumbs list with junk really quickly.

The problem is that the addEventListener monkeypatch wraps every listener function but only specifies a before breadcrumb handler for click and keypress event types. However, wrap will automatically reuse a previous wrapper for a function with no regard for differences in the _before argument (bug apparently introduced in commit 99cf147. So if click wraps a function first then all events will generate breadcrumbs, but if another event type wraps the function first then click won't generate breadcrumbs either.

This issue is particularly acute with jQuery, which uses a single event handling function per element for all event types, but applies to any code that attaches the same listener function to both click and other events.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions