Page MenuHomePhabricator

Invalid value None for integer property "action.abort.timing" (schema:Edit)
Closed, ResolvedPublic1 Estimated Story Points

Description

Currently trending at https://logstash.wikimedia.org/#/dashboard/elasticsearch/eventlogging-errors with over 1100 hits in the past 24 hours (which are silently dropped from further processing).

event revision 13457736
event schema Edit
message level ERROR
message

None is not of type u'integer'

See also https://meta.wikimedia.org/wiki/Schema:Edit

Sample:

{"event":{"version":1,"action":"abort","editor":"visualeditor","platform":"desktop","integration":"page",..,"user.id":0,"user.editCount":0,"mediawiki.version":"1.27.0-wmf.3","user.class":"IP","action.abort.type":"nochange","action.abort.mechanism":"navigate-back","action.abort.timing":null},"revision":13457736,"schema":"Edit","webHost":"vi.wikipedia.org","wiki":"viwiki"}

Presumably the error is about property action.abort.timing being null.

Event Timeline

Krinkle raised the priority of this task from to Needs Triage.
Krinkle updated the task description. (Show Details)
Krinkle added subscribers: Jdforrester-WMF, Krinkle, Aklapper.

Yes, this is action.abort.timing being set to NaN by JS which then gets changed to null when converting to JSON, and then None when being read by Python.

I guess this happens when you try to abort with no change before VE is ready.

Which should really be type preinit instead of nochange

Change 249143 had a related patch set uploaded (by Jforrester):
mw.trackSubscriber: Handle timeStamp being undef/NaN/infinite

https://gerrit.wikimedia.org/r/249143

Change 249149 had a related patch set uploaded (by Krinkle):
mw.trackSubscriber: If the number is NaN/falsy, return -1;

https://gerrit.wikimedia.org/r/249149

Jdforrester-WMF triaged this task as High priority.
Jdforrester-WMF edited a custom field.

What's the status here? Above patches not merged.

Looks like this was fixed months ago, sorry.

Change 249149 abandoned by Jforrester:
mw.trackSubscriber: If the number is NaN/falsy, return -1;

Reason:
Fixed elsewise.

https://gerrit.wikimedia.org/r/249149

Change 249143 abandoned by Jforrester:
mw.trackSubscriber: Handle timeStamp being undef/NaN/infinite

Reason:
Fixed elsewise.

https://gerrit.wikimedia.org/r/249143

nshahquinn-wmf raised the priority of this task from High to Needs Triage.Mar 30 2018, 10:31 AM
nshahquinn-wmf moved this task from Backlog to Radar on the Contributors-Analysis board.