Page MenuHomePhabricator

AbuseFilter does not report user_group or user_name when CreateLocalAccount is performed
Open, Needs TriagePublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

  • Implement an abuse filter to prevent creation of certain account name on Chinese Wikipedia (see AF194)
  • Use Special:CreateLocalAccount to create a user name that would be blocked by the abuse filter

What happens?:
The AbuseLog does not report the user performing the CreateLocalAccount action nor the group the user is in.

What should have happened instead?:
AbuseFilter would report user_group so the filter would allow admins or other permitted user to create a local account where it was a false positive and a rewrite of the filtering rules is not feasible..

Software version (on Special:Version page; skip for WMF-hosted wikis like Wikipedia):

Other information (browser name/version, screenshots, etc.):

Event Timeline

Restricted Application added subscribers: Stang, Aklapper. · View Herald Transcript
Tgr subscribed.

The amount of details reported seems pretty minimal: https://zh.wikipedia.org/wiki/Special:%E6%BB%A5%E7%94%A8%E6%97%A5%E5%BF%97/5138031

I think this is an issue with AbuseFilter, or the AbuseFilter-related hook handlers in CentralAuth, not CentralAuth itself. The code triggering the autocreation is in CentralAuthForcedLocalCreationService.

T307827 mentions AF194, too. So I think it is the same problem.

Change #1194680 had a related patch set uploaded (by Dragoniez; author: Dragoniez):

[mediawiki/core@master] AuthManager: Include $performer in options for testUserForCreation

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

Change #1194682 had a related patch set uploaded (by Dragoniez; author: Dragoniez):

[mediawiki/extensions/AbuseFilter@master] [WIP] Compute user_* variables for forced autocreation via Special:CreateLocalAccount

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

Dragoniez subscribed.

Removing Chinese-Sites because this (annoying) issue isn't limited to Chinese sites.

Change #1194680 merged by jenkins-bot:

[mediawiki/core@master] AuthManager: Include $performer in options for testUserForCreation

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