Inconsistent FK ON DELETE behavior: bulk_operations.performed_by and ticket_templates.created_by default to RESTRICT #40

New Issue

2026-08-31T21:29:46-04:00

jared commented

2026-08-31 21:29:46 -04:00

Severity: Medium

In migrations/000_baseline.sql, tickets.created_by/updated_by/assigned_to, ticket_attachments.uploaded_by, ticket_dependencies.created_by, recurring_tickets.created_by/assigned_to, and api_keys.created_by all use ON DELETE SET NULL. But bulk_operations.performed_by (~line 72) and ticket_templates.created_by (~line 241) have no ON DELETE clause, which defaults to RESTRICT.

Impact: Deleting a user who ever ran a bulk operation or created a template will hard-fail at the DB level instead of nulling out the reference, breaking the pattern used everywhere else in the schema and potentially blocking legitimate user offboarding/cleanup.

Fix: Add a migration to change both FKs to ON DELETE SET NULL, matching the rest of the schema.

**Severity:** Medium In `migrations/000_baseline.sql`, `tickets.created_by/updated_by/assigned_to`, `ticket_attachments.uploaded_by`, `ticket_dependencies.created_by`, `recurring_tickets.created_by/assigned_to`, and `api_keys.created_by` all use `ON DELETE SET NULL`. But `bulk_operations.performed_by` (~line 72) and `ticket_templates.created_by` (~line 241) have no `ON DELETE` clause, which defaults to `RESTRICT`. **Impact:** Deleting a user who ever ran a bulk operation or created a template will hard-fail at the DB level instead of nulling out the reference, breaking the pattern used everywhere else in the schema and potentially blocking legitimate user offboarding/cleanup. **Fix:** Add a migration to change both FKs to `ON DELETE SET NULL`, matching the rest of the schema.

jared added the data-integrity priority/medium labels 2026-09-08 10:15:44 -04:00

jared referenced this issue from a commit

2026-09-08 11:59:38 -04:00

Fix inconsistent FK ON DELETE behavior on bulk_operations/ticket_templates (#40)

jared referenced this issue from a commit

2026-09-08 15:39:42 -04:00

Merge development into main: 15-issue triage + fix batch

Sign in to join this conversation.