ampproject / error-reporting

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

🚨 Error: Assertion failed #63

Open ampprojectbot opened 3 years ago

ampprojectbot commented 3 years ago

Details

Error report: link First seen: Mar 4, 2021 Frequency: ~ 1,104/day

Stacktrace

Error: Assertion failed
    at message (src/core/assert.js:92:29)
    at Error (src/core/assert.js:180:4)
    at parentResource (src/service/owners-impl.js:118:16)
    at element (src/service/owners-impl.js:117:22)
    at findResourcesInElements_ (src/service/owners-impl.js:165:4)
    at scheduleLayoutOrPreloadForSubresources_ (src/service/owners-impl.js:63:4)
    at extensions/amp-consent/0.1/consent-ui.js:383:8
    at extensions/amp-consent/0.1/amp-consent.js:178:6
    at src/custom-element.js:1493:8
    at src/service/resource.js:1109:4
    at r (src/service/resources-impl.js:1832:37)
    at r (src/service/resources-impl.js:1832:31)
    at callback (src/finite-state-machine.js:72:6)
    at src/service/resources-impl.js:748:42
    at src/service/resources-impl.js:136:42
    at src/pass.js:109:4
    at src/pass.js:55:6
    at callback (src/service/timer-impl.js:79:10)

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:165-165 in #27350 (Mar 30, 2020) @powerivq modified src/service/owners-impl.js:44-74 in #23668 (Aug 26, 2019) @dvoytenko modified extensions/amp-consent/0.1/consent-ui.js:366-392 in #32480 (Feb 8, 2021) @dvoytenko modified extensions/amp-consent/0.1/amp-consent.js:168-181 in #32480 (Feb 8, 2021) @dvoytenko modified src/custom-element.js:1493-1493 in #32345 (Feb 4, 2021) @dvoytenko modified src/service/resource.js:1105-1118 in #3544 (Jun 14, 2016) @renovate-bot modified src/service/resources-impl.js:1832-1832 in #27350 (Mar 30, 2020) @renovate-bot modified src/service/resources-impl.js:1832-1832 in #27350 (Mar 30, 2020) @jridgewell modified src/finite-state-machine.js:63-78 in #1637 (Mar 18, 2016) @dvoytenko modified src/service/resources-impl.js:748-748 in #24566 (Sep 20, 2019) @jridgewell modified src/service/resources-impl.js:136-136 in #9789 (Jun 8, 2017) @jridgewell modified src/pass.js:105-112 in #3513 (Jun 9, 2016) @rsimha modified src/pass.js:50-56 in #23559 (Jul 29, 2019) @rsimha modified src/service/timer-impl.js:73-81 in #21212 (May 16, 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.