Page MenuHomePhabricator

Remove explicit administrative actions from the user activity log
ClosedPublic

Authored by epriestley on Jul 19 2019, 6:17 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 14, 5:36 PM
Unknown Object (File)
Feb 3 2024, 10:58 PM
Unknown Object (File)
Jan 25 2024, 1:02 AM
Unknown Object (File)
Dec 18 2023, 2:39 PM
Unknown Object (File)
Nov 29 2023, 11:05 PM
Unknown Object (File)
Nov 25 2023, 10:21 PM
Unknown Object (File)
Nov 25 2023, 10:20 PM
Unknown Object (File)
Nov 25 2023, 10:20 PM
Subscribers
None

Details

Summary

Depends on D20669. Ref T13343. Currently, the user activity log includes a number of explicit administrative actions which some administrator (not a normal user or a suspicious remote address) takes. In most/all cases, these changes are present in the user profile transaction log too, and that's generally a better place for them (for example, it doesn't get GC'd after a couple months).

Some of these are so old that they have no writers (like DELETE and EDIT). I'd generally like to modernize this a bit so we can reference it in email (see T13343) and I'd like to modularize the event types as part of that -- partly, cleaning this up makes that modularization easier.

There's maybe some hand-wavey argument that administrative vs non-administrative events could be related and might be useful to see in a single log, but I can't recall a time when that was actually true, and we could always build that kind of view later by just merging the two log sources, or by restoring double-writes for some subset of events. In practice, I've used this log mostly to look for obvious red flags when users report authentication difficulty (e.g., many unauthorized login attempts), and removing administrative actions from the log is only helpful in that use case.

Test Plan

Grepped for all the affected constants, no more hits in the codebase.

Diff Detail

Repository
rP Phabricator
Branch
elogin9
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 23164
Build 31817: Run Core Tests
Build 31816: arc lint + arc unit