Page MenuHomePhabricator

Wikimedia global user rights log entry did not get updated in global rename
Closed, ResolvedPublic

Event Timeline

Krenair assigned this task to Legoktm.
Krenair raised the priority of this task from to Needs Triage.
Krenair updated the task description. (Show Details)
Krenair added a project: MediaWiki-User-rename.
Krenair changed Security from none to None.
Krenair added subscribers: Krenair, Spage, Jalexander.

So I think this is actually a larger question with global renames. We had the same issue with a globally locked account being renamed. The lock itself was transferred from Account A to Account A (Usurped) but the log entry did not.

Example:
https://meta.wikimedia.org/w/index.php?title=Special:CentralAuth&target=Commons%20fair%20use%20upload%20bot original account name with lock log (now owned by someone else)

https://meta.wikimedia.org/w/index.php?title=Special%3ACentralAuth&target=Commons+fair+use+upload+bot+%28usurped%29 usurped account with the lock correctly transferred but no log entry.

Aklapper triaged this task as Medium priority.Dec 22 2014, 5:14 PM
Glaisher subscribed.

This is an issue with local renames itself.
When doing a rename:

UPDATE `logging` SET log_title = 'NewUser' WHERE log_type IN ('block','newusers','rights','renameuser') AND log_namespace = '2' AND log_title = 'OldUser'

and those log_types are from SpecialLog::getLogTypesOnUser() which doesn't know about CentralAuth's logs. It does have GetLogTypesOnUser hook but the problem is CentralAuth's logs are too weird. In gblrights, all changes to global groups and wikisets are also logged in addition to global user rights changes. In globalauth log also we've those @global stuff. I guess we can't use that hook here.

Why global renamers is not a global group??

Why global renamers is not a global group??

We don't want people to be able to globally rename from every wiki.

We don't want people to be able to globally rename from every wiki.

That is somehow a strange interpretation what is a global group, to see a "unconfirmed" user in local logs doing this. This group has only global change rights. Than a proper name would be:
Hidden global group

dmehus added a subscriber: Reception123.

Adding User-RhinosF1 project and moving on that workboard to Miraheze Linked, as we're quite curious when this bug will be fixed. All local log entries have the performers and targets updated following a global rename, so why not global log entries, such as the global rights log?

Adding @Reception123 per request on IRC.

Change 713075 had a related patch set uploaded (by Zabe; author: Zabe):

[mediawiki/extensions/CentralAuth@master] Add onGetLogTypesOnUser Hook in order to treat 'glbrights' better

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

Change 713075 merged by jenkins-bot:

[mediawiki/extensions/CentralAuth@master] Add onGetLogTypesOnUser Hook in order to treat 'gblrights' better

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

taavi assigned this task to Zabe.

Change 714150 had a related patch set uploaded (by RhinosF1; author: Zabe):

[mediawiki/extensions/CentralAuth@REL1_36] Add onGetLogTypesOnUser Hook in order to treat 'gblrights' better

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

Change 714150 merged by jenkins-bot:

[mediawiki/extensions/CentralAuth@REL1_36] Add onGetLogTypesOnUser Hook in order to treat 'gblrights' better

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