Page MenuHomePhabricator

TypeError: Cannot read property 'key' of undefined
Closed, DeclinedPublicPRODUCTION ERROR

Description

Error
normalized_message
TypeError: Cannot read property 'key' of undefined
exception.trace
at setStatus  <anonymous>:35:65
at Object.setBannerShown  <anonymous>:38:528
at Object.reallyInsertBanner  <anonymous>:27:174
at HTMLDocument.<anonymous>  <anonymous>:28:627
Impact

111 occurrences in past 12 hrs

Notes

Looks like occurrences are coming from ru.wikipedia.org, and started around

Stack trace shows error originating from here
https://gerrit.wikimedia.org/g/mediawiki/extensions/CentralNotice/+/420a5aa86158fcbc43810d142c0bd453f4f72b25/resources/ext.centralNotice.display/state.js#546

Logstash url: https://logstash.wikimedia.org/goto/e008b5edcf03b4cf67d46c8d8210cc11

Event Timeline

Krinkle updated the task description. (Show Details)

Perhaps this was an error in some in-banner JS? There haven't been any more occurrences of it in the past 12 days, and I don't see the CentralNotice code in that path trying to access any property called 'key'.

Krinkle subscribed.

The stack from setBannerShown() onwards is in built-in CentralNotice code and takes no parameters, so the calls after that should not be able to crash based on banner content scripts, but I agree it's likely that a banner or URL parameter did cause it. Let's close for now until it either becomes significant in frequency, or reproducible, or correlated with a user-facing issue.