Page MenuHomePhabricator

Wrong AF variables computed for MassMessage account for edits on Flow topics
Closed, ResolvedPublic

Description

See for example: https://www.mediawiki.org/wiki/Special:AbuseFilter/examine/log/80822

  • Observed: user_name - 127.0.0.1 user_editcount - null
  • Expected: user_name - MediaWiki message delivery user_editcount - whatever is the actual editcount

Event Timeline

Glaisher raised the priority of this task from to Needs Triage.
Glaisher updated the task description. (Show Details)
Glaisher added a subscriber: Glaisher.
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Legoktm triaged this task as High priority.Oct 19 2015, 4:55 PM
Legoktm added a subscriber: Legoktm.

Flow reads and stores the value of $wgUser during $wgExtensionFunctions in FlowHooks::getAbuseFilter(), which is way too early.

Change 247301 had a related patch set uploaded (by Legoktm):
AbuseFilter: Get user from context instead of at initialization

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

Change 247301 merged by jenkins-bot:
AbuseFilter: Get user from context instead of at initialization

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

Looks like this is still not fixed...

It's actually fixed. https://www.mediawiki.org/wiki/Special:AbuseFilter/examine/2140225 I guess I updated the filter before this reached mediawiki.org causing it to be blocked again.