Page MenuHomePhabricator

Make CheckUser record the log_id of logged actions
Open, HighPublic

Description

Problem.
As mentioned in T252485#6155656 there isn't a way to link to a specific logged action within CheckUser

Proposed Solution
Record the log_id in the cu_changes table so users can get a link to the specific line in the log.

Related Objects

StatusSubtypeAssignedTask
OpenNone
ResolvedSecurityZabe
ResolvedSecurityDreamy_Jazz
ResolvedSecurityDreamy_Jazz
ResolvedSecurityDreamy_Jazz
OpenBUG REPORTNone
OpenBUG REPORTNone
OpenBUG REPORTNone
OpenDreamy_Jazz
OpenFeatureDreamy_Jazz
ResolvedLadsgroup
ResolvedMilimetric
OpenNone
OpenNone
OpenNone
OpenDreamy_Jazz
OpenNone
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedMarostegui
ResolvedPRODUCTION ERRORDreamy_Jazz
ResolvedMarostegui
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedBUG REPORTDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedBUG REPORTDreamy_Jazz
ResolvedBUG REPORTDreamy_Jazz
OpenDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedPRODUCTION ERRORpmiazga
OpenNone
ResolvedDreamy_Jazz
OpenNone
ResolvedDreamy_Jazz
OpenNone

Event Timeline

I feel like we had a separate task for it already, but cannot find it.

Anyway, one immediate issue that comes to mind is that log_id alone is not sufficient and some indication of log type (or source) should be included as well. This is because not all logs are stored in the logging table. The most obvious example is AbuseFilter logs. Other extensions may also have their own logs and want to expose them in CheckUser (e.g. Thanks althought we have not integrated it yet, or MediaWiki-extensions-LoginNotify as discussed in T174492)

Aklapper renamed this task from Should CheckUser record the log_id of logged actions? to Make CheckUser record the log_id of logged actions.Sep 21 2022, 11:25 AM
Dreamy_Jazz removed a subtask: Restricted Task.
Dreamy_Jazz added a parent task: Restricted Task.

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

[mediawiki/extensions/CheckUser@master] Add column 'cuc_log_id' to table 'cu_changes'

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

Change 856930 abandoned by Dreamy Jazz:

[mediawiki/extensions/CheckUser@master] Add column 'cuc_log_id' to table 'cu_changes'

Reason:

In favour of Ied5b0ccbfd73751f9a47674f2e7e98efa469d568

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