search

ampproject / error-reporting

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

🚨 Error: Assertion failed #60

Open ampprojectbot opened 3 years ago

ampprojectbot commented 3 years ago

Details

Error report: link First seen: Mar 4, 2021 Frequency: ~ 3,213/day

Stacktrace

Error: Assertion failed
    at src/core/assert.js:92:16
    at assertion (src/core/assert.js:179:9)
    at devAssert (src/service/owners-impl.js:118:6)
    at forEach (src/service/owners-impl.js:117:13)
    at findResourcesInElements_ (src/service/owners-impl.js:75:4)
    at schedulePause (extensions/amp-consent/0.1/consent-ui.js:369:56)
    at pause (extensions/amp-consent/0.1/consent-ui.js:323:9)
    at hide (extensions/amp-consent/0.1/amp-consent.js:914:27)
    at callback (src/service/vsync-impl.js:469:16)

Notes

@rcebulko modified src/core/assert.js:92-93 in #32887 (Feb 25, 2021) @rcebulko modified src/core/assert.js:99-193 in #32788 (Feb 23, 2021) @lannka modified src/service/owners-impl.js:118-155 in #24222 (Aug 30, 2019) @renovate-bot modified src/service/owners-impl.js:117-117 in #27350 (Mar 30, 2020) @renovate-bot modified src/service/owners-impl.js:75-75 in #27350 (Mar 30, 2020) @dvoytenko modified extensions/amp-consent/0.1/consent-ui.js:366-392 in #32480 (Feb 8, 2021) @dvoytenko modified extensions/amp-consent/0.1/consent-ui.js:323-324 in #32480 (Feb 8, 2021) @zhouyx modified extensions/amp-consent/0.1/amp-consent.js:914-914 in #18211 (Sep 20, 2018) @jridgewell modified src/service/vsync-impl.js:469-470 in #20836 (Feb 14, 2019)

Seen in:

Possible assignees: @rcebulko, @dvoytenko

/cc @ampproject/release-on-duty

rcebulko commented 3 years ago

As a result of https://github.com/ampproject/amphtml/pull/32828 , devAsserts temporarily made it into Nightly build. They are being removed again (fix in https://github.com/ampproject/amphtml/pull/33087) but this was one assertion which was failing in production. Suggest replacing this devAssert with explicit error report to collect data.