Skip to content

[HIGH][SECURITY] Crypto tamper check fails open when verification fails (scope: cache present but invalid) #62

Description

@Aniketvish0

Migrated from armoriq/conmap-auto#713.

Fix lands in: armoriq/armorclaude and armoriq/armorcodex. Filed here because conmap-auto is this session's tracker for CLI/SDK work.

Line refs from origin/main: armorclaude @ 688bbc3 (AC), armorcodex @ 72d9e13 (AX).

AC engine.mjs:879-893 and AX engine.mjs:339-354 — the integrity check catches, writes a debugLog (off by default, see the companion debug-logging issue) and continues.

Separately the check is skipped altogether when the digest cache is missing, and that file is trivially deletable.

Ask

  • Fail closed.
  • Treat a missing digest cache as a failed verification, not a skip.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingsecuritySecurity vulnerability or concern

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions