search

ampproject / error-reporting

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

🚨 Error: Assertion failed #64

Open ampprojectbot opened 3 years ago

ampprojectbot commented 3 years ago

Details

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

Stacktrace

Error: Assertion failed
    at src/core/assert.js:92:16
    at assertion (src/core/assert.js:179:9)
    at devAssert (src/utils/math.js:98:2)
    at getResponsiveHeight_ (extensions/amp-ad-network-adsense-impl/0.1/responsive-state.js:422: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) @rcebulko modified src/utils/math.js:98-98 in #21876 (May 21, 2019) @Jiaming-X modified extensions/amp-ad-network-adsense-impl/0.1/responsive-state.js:419-428 in #28831 (Jun 11, 2020)

Seen in:

Possible assignees: @rcebulko, @Jiaming-X

/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.