Page MenuHomePhabricator

Enable read new support in Special:CheckUser's 'Get IP Addresses' mode
Closed, ResolvedPublic

Description

Now that CheckUser writes to the new tables for event table migration (T330158), reading from these tables when the migration stage is set to read new can commence.

This task is to add support to read new in Special:CheckUser's 'Get IP addresses' mode.

Acceptance criteria
  • Alias the current query info fields in CheckUserGetIPsPager to prepare for migration
  • Add code to read from the two new tables when reading new in CheckUserGetIPsPager

QA Results - Local

Event Timeline

Change 884133 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Prepare CheckUserGetIPsPager for event table migration

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

Change 884133 abandoned by Dreamy Jazz:

[mediawiki/extensions/CheckUser@master] Prepare CheckUserGetIPsPager for event table migration

Reason:

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

Change 938229 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Implement read new support for CheckUserGetIPsPager

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

Change 954948 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] [WIP] Implement read new support for CheckUserGetIPsPager

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

Change 938229 abandoned by Dreamy Jazz:

[mediawiki/extensions/CheckUser@master] [WIP] Implement read new support for CheckUserGetIPsPager

Reason:

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

Dreamy_Jazz renamed this task from Use the CheckUserUnionSelectQueryBuilder in Special:CheckUser's 'Get IP Addresses' mode to Enable read new support in Special:CheckUser's 'Get IP Addresses' mode.Sep 6 2023, 7:59 PM
Dreamy_Jazz updated the task description. (Show Details)

Change 954948 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Implement read new support for CheckUserGetIPsPager

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

@Dreamy_Jazz Get IP address works as designed as seen in the screenshots. The task will be moved to Done. Thanks for all your work!

Status: ✅PASS
Environment: Local: 1.42.0-alpha (a7690c5)14:05, 12 December 2023
OS: macOS Sonoma 14.2
Browser: Chrome 120, Firefox 119, Safari 17.0
Skins. Vector 2022, 2010, Minerva, Monobook, Timeless
Device: MBA M2
Emulated Device:: n/a
Test Links:
http://localhost:8080/wiki/Special:CheckUser

✅AC1: https://phabricator.wikimedia.org/T328998

Different IPsGet Edit XFFTemp UserHebrewMinerva- different size
2023-12-12_09-31-52.png (684×1 px, 118 KB)
2023-12-12_09-32-09.png (988×1 px, 206 KB)
2023-12-12_10-13-23.png (944×1 px, 120 KB)
2023-12-12_17-03-42.png (1×2 px, 211 KB)
2023-12-12_17-08-33.png (1×590 px, 161 KB)

Thanks for the QA! Marking this as resolved as these changes are now used on all wikis.