cipher: encrypt_text_at_rest silently stores empty ciphertext on failure (data loss) #162

Closed
opened 2026-07-27 01:28:26 +00:00 by KofTwentyTwo · 1 comment
KofTwentyTwo commented 2026-07-27 01:28:26 +00:00 (Migrated from github.com)

Found in the 0.E security review. PayloadCipher::encrypt_text_at_rest uses unwrap_or_default() on encrypt_bytes, so an encryption failure silently persists empty ciphertext (data loss, not a key bypass). Pre-existing. Fix: propagate the error. area: store.

Found in the 0.E security review. `PayloadCipher::encrypt_text_at_rest` uses `unwrap_or_default()` on `encrypt_bytes`, so an encryption failure silently persists empty ciphertext (data loss, not a key bypass). Pre-existing. Fix: propagate the error. area: store.
KofTwentyTwo commented 2026-07-28 14:06:20 +00:00 (Migrated from github.com)

Superseded by #198 (M6) in the roadmap restructuring (milestones M1–M7). The replacement story carries the full, self-contained spec for a remote implementer. See docs/ROADMAP.md, docs/STORY-WORKFLOW.md, docs/HANDOFF.md.

Superseded by #198 (M6) in the roadmap restructuring (milestones M1–M7). The replacement story carries the full, self-contained spec for a remote implementer. See docs/ROADMAP.md, docs/STORY-WORKFLOW.md, docs/HANDOFF.md.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: KofTwentyTwo/nuncio#162