Page MenuHomePhabricator

Special:Investigate timeline shows user links for the checkuser's IP instead of the investigated IP
Closed, ResolvedPublic

Description

This is due to TimelineRowFormatter calling User::newFromId(0), which returns a user with the global user's IP as the name.

Called from: https://github.com/wikimedia/mediawiki-extensions-CheckUser/blob/master/src/TimelineRowFormatter.php#L333

Thanks @Urbanecm for finding this.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Thanks @Bsadowski1 for noticing the bug _and_ fixing the cause, I'm just a postman and patchauthor :-).

Change 633006 had a related patch set uploaded (by Urbanecm; owner: Urbanecm):
[mediawiki/extensions/CheckUser@master] TimelineRowFormatter::getUserLinks should not assume the user is registered

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

Change 633006 merged by jenkins-bot:
[mediawiki/extensions/CheckUser@master] TimelineRowFormatter::getUserLinks should not assume the user is registered

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

dom_walden mentioned this in Unknown Object (Task).Oct 9 2020, 7:24 AM

Moving to Done since @dom_walden has tested: T265103#6531647