webhooker/internal/database/retention_test.go at issue-115-mask-http-destination-url

Files

clawbot e50a79ced9

check / check (push) Superseded by a newer commit; never tested

Allow retention_days of 0 to mean retain forever (closes #79 )

Rewrites retention_days=0 to the RetentionForeverDays sentinel (365 * 1000)
in Webhook.BeforeSave, so the GORM column default cannot win the race. The
reaper skips retain-forever webhooks before building any query.

Also bounds the reaper's cutoff arithmetic: a time.Duration is int64
nanoseconds, so day counts above MaxFiniteRetentionDays (106751) overflowed
and wrapped the cutoff into the future, where created_at &lt; cutoff matched
every row and the sweep deleted everything. parseRetentionDays now rejects
finite values above the ceiling, and retentionCutoff saturates so rows
written by older versions cannot reach it either.

Views render RetentionLabel() rather than the raw sentinel.

2026-08-11 14:35:34 +02:00

9.2 KiB

Raw Permalink Blame History

View Raw

9.2 KiB Raw Permalink Blame History

9.2 KiB

Raw Permalink Blame History