Page MenuHomePhabricator

Add a GC for user logs
ClosedPublic

Authored by epriestley on Feb 1 2014, 11:23 PM.
Tags
None
Referenced Files
F18658017: D8128.id18384.diff
Sep 23 2025, 4:29 AM
F18609165: D8128.diff
Sep 14 2025, 2:28 AM
F18087765: D8128.id18384.diff
Aug 6 2025, 5:23 AM
F18083627: D8128.id18398.diff
Aug 5 2025, 10:54 AM
F18053378: D8128.id.diff
Aug 4 2025, 4:42 AM
F18051875: D8128.diff
Aug 4 2025, 12:38 AM
F17781023: D8128.id18384.diff
Jul 24 2025, 5:29 AM
F17755865: D8128.diff
Jul 22 2025, 1:18 PM
Subscribers

Details

Reviewers
btrahan
Maniphest Tasks
T4368: Implement GC for mail
Commits
Restricted Diffusion Commit
rP5a9a96735dde: Add a GC for user logs
Summary

Fixes T4368. This is the last "obvious" table we have which we should be GC'ing but do not. It's about 1/12th of the data on secure.phabricator.com.

This table stores logins, account creation, password resets, login attempts, etc, and is primarily useful if something sketchy happens so you can go back and review login activity. This data is not useful indefinitely, and there's no reason to retain it forever. Because you don't always know when something sketchy happened I've given this table a fairly long TTL (180 days), but we don't need limitless amounts of this data.

Test Plan

Ran phd debug garbage and saw a reasonable amount of data get GC'd. This table already has an appropriate key.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped