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

Beacon script can be injected multiple times when there are multiple </body> in the page. #6960

Open
MathieuLamiot opened this issue Sep 8, 2024 · 0 comments · May be fixed by #7231
Open
Assignees
Labels
module: Beacon priority: low Issues that can wait quick win severity: minor Defect that does not affect functionality
Milestone

Comments

@MathieuLamiot
Copy link
Contributor

Context

https://wp-media.slack.com/archives/CUT7FLHF1/p1725795776110169?thread_ts=1725624439.071799&cid=CUT7FLHF1

The impact is that the beacon executes multiple times, and performs multiple AJAX calls. Only the first one is useful.

Expected behavior

The beacon is injected at most once.

Acceptance Criteria
On a page with multiple , the beacon is injected only once.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module: Beacon priority: low Issues that can wait quick win severity: minor Defect that does not affect functionality
Projects
None yet
3 participants