Skip to content

feat(coderd/database/dbpurge): make API keys retention configurable #21037

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mafredri merged 6 commits into from
Dec 2, 2025
Merged

feat(coderd/database/dbpurge): make API keys retention configurable #21037

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
5e8003f
feat(coderd/database/dbpurge): make API keys retention configurable
mafredri Dec 1, 2025
a8c815d
fix: improve comment wording for API keys retention
mafredri Dec 1, 2025
85235cc
test(coderd/database/dbpurge): convert API keys tests to table-driven
mafredri Dec 1, 2025
1a09ab4
chore: remove global retention fallback for API keys
mafredri Dec 2, 2025
a6744f0
refactor: optimize DeleteExpiredAPIKeys to use :execrows
mafredri Dec 2, 2025
87f1ce1
Merge branch 'main' into mafredri/feat-coderd-db-retention-policy-4
mafredri Dec 2, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: improve comment wording for API keys retention
  • Loading branch information
@mafredri
mafredri committed Dec 2, 2025
commit a8c815db3f719d30f9332adf95e99fd1f2a8407b
4 changes: 2 additions & 2 deletions coderd/database/dbpurge/dbpurge.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,8 @@ func New(ctx context.Context, logger slog.Logger, db database.Store, vals *coder
}
if apiKeysRetention > 0 {
// Delete keys that have been expired for at least the retention period.
// This allows the backend to return a more helpful error when a user
// tries to use an expired key.
// A higher retention period allows the backend to return a more helpful
// error message when a user tries to use an expired key.
deleteExpiredKeysBefore := start.Add(-apiKeysRetention)
expiredAPIKeys, err = tx.DeleteExpiredAPIKeys(ctx, database.DeleteExpiredAPIKeysParams{
Before: dbtime.Time(deleteExpiredKeysBefore),
Expand Down