search

ampproject / error-reporting

Contains production error tracking issues.
Apache License 2.0
2 stars 5 forks source link

🚨 Error: Attempting to trigger event for detached target: %s [object HTMLElement] #54

Closed ampprojectbot closed 3 years ago

ampprojectbot commented 3 years ago

Details

Error report: link First seen: Feb 13, 2021 Frequency: ~ 27/day

Stacktrace

Error: Attempting to trigger event for detached target: %s [object HTMLElement]
    at Wa (https://cdn.ampproject.org/rtv/042102130314000/v0.mjs:2:10961)
    at Va.La (https://cdn.ampproject.org/rtv/042102130314000/v0.mjs:2:9183)
    at Va.error (https://cdn.ampproject.org/rtv/042102130314000/v0.mjs:2:9281)
    at ae.triggerEventForTarget (amp-analytics-latest.mjs:4:62749)
    at amp-ad-network-doubleclick-impl-0.1.mjs:4:23613

/cc @ampproject/release-on-duty

rcebulko commented 3 years ago

/cc @jridgewell @erwinmombay This is a new error that started popping up recently on Android browsers. It looks like sourcemapping ESM might be acting up?

jridgewell commented 3 years ago

@samouri This error is getting huge numbers.

ampprojectbot commented 3 years ago

A duplicate error report was linked to this issue (link)

ampprojectbot commented 3 years ago

A duplicate error report was linked to this issue (link)

samouri commented 3 years ago

I'm scratching my head as to how it is possible that https://github.com/ampproject/amphtml/pull/32637 is creating so many errors, and especially that the errors weren't already preexisting. I would have expected this to be the new name for the __AMP_TOP error.

Reports are all coming from doubleclick ads.

I went to example sites and wasn't able to reproduce. On a hunch, I tested both adblock plus and ublock origin, both of which didn't reproduce this error.

samouri commented 3 years ago

Two leftover weak hunches:

  1. doubleclick expects to report on detached elements somehow, and it hadn't been throwing errors.
  2. a different change in the same release window is causing this. no commits look suspicious though.
ampprojectbot commented 3 years ago

A duplicate error report was linked to this issue (link)

ampprojectbot commented 3 years ago

A duplicate error report was linked to this issue (link)

ampprojectbot commented 3 years ago

A duplicate error report was linked to this issue (link)

ampprojectbot commented 3 years ago

A duplicate error report was linked to this issue (link)

ampprojectbot commented 3 years ago

A duplicate error report was linked to this issue (link)

samouri commented 3 years ago

This should be solved in next week's stable push