Page MenuHomePhabricator

Duration of query recent changes in Check user tool
Open, MediumPublic

Description

Hello,

Can we add the selected duration to the check user log?

For example, if I selected to check 0.0.0.0/16 recent changes in (last week or last two weeks or last 30 days or all), then in the log it'll appear like this:

03:40, 14 September 2018, alanajjar got users for 0.0.0.0/16 (reason)

So, I think it'll be better, if we add the duration, to be like this:

03:40, 14 September 2018, alanajjar got users in last week for 0.0.0.0/16 (reason)

Or

03:40, 14 September 2018, alanajjar got users for 0.0.0.0/16 (reason) - last 30 days

Benefits
It'll help to follow any CU abuse or misuse. As you know, if the duration changed then the results will changed too.
Also, I think every inputs in CU tool should appear in the log.


Is it controversial proposal? I think no.

Event Timeline

alaa created this task.Sep 14 2018, 12:42 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 14 2018, 12:42 PM

So, to clarify, you want that when somebody checks an IP or user not for the whole standard duration but for a limited period of time (last week, last two weeks, etc.), the logs say so. Right?

alaa added a comment.Sep 15 2018, 6:48 PM

So, to clarify, you want that when somebody checks an IP or user not for the whole standard duration but for a limited period of time (last week, last two weeks, etc.), the logs say so. Right?

Exactly

Change 464472 had a related patch set uploaded (by MR70; owner: MR70):
[mediawiki/extensions/CheckUser@master] [WIP] Show duration of query recent changes in checkuser

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

MR70 triaged this task as High priority.Oct 4 2018, 9:04 PM
MR70 added a project: DBA.
MR70 moved this task from Backlog to Patches in review on the CheckUser board.
MR70 added a subscriber: Huji.
Marostegui added a subscriber: Marostegui.

I don't think there's anything to do for the DBAs here

Change 479151 had a related patch set uploaded (by MR70; owner: MR70):
[mediawiki/extensions/CheckUser@master] [WIP] Show duration of query recent changes in checkuser

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

Change 464472 abandoned by MR70:
[WIP] Show duration of query recent changes in checkuser

Reason:
merge-conflict issue

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

Hi @Huji , Could you please take a look at the commit ?

Jalexander lowered the priority of this task from High to Medium.Dec 12 2018, 7:54 PM

Change 479151 abandoned by WMFOffice:
Show the queried duration in CheckUser logs

Reason:
abandoning patch sets of globally banned user

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

Meno25 added a subscriber: Meno25.Mar 23 2019, 8:30 PM
Bradv added a subscriber: Bradv.Oct 8 2020, 9:04 PM