Skip to content

break-glass: integrity check over plain HTTP; release record panel gap and strip - #82

Merged
jfillman merged 1 commit into
mainfrom
breakglass-ui-fixes
Oct 12, 2026
Merged

jfillman merged 1 commit into
mainfrom
breakglass-ui-fixes

Conversation

@jfillman

Copy link
Copy Markdown
Owner

Two things found in the browser on 2026-10-12.

  • "integrity not checked" on every recording. crypto.subtle exists only in a secure context (HTTPS or localhost); this Backstage is served over plain HTTP, so the player could not hash. sha256Hex now falls back to a plain-JS SHA-256 (tested against the FIPS 180-4 vectors, the 55/64-byte padding boundary and a million bytes, and against Node's digest). The SHA-256 matches the session record chip now appears everywhere.
  • Release record panel. Break-glass sessions during this release hugged the columns above it and had no colour strip; it now keeps the same 18 px gap as the Human context panel and carries a strip in the break-glass red.

🤖 Generated with Claude Code

…d's break-glass panel gets its gap and strip

- RecordingPlayer said 'integrity not checked' for every recording: crypto.subtle exists only in a secure context and
  this Backstage is served over plain HTTP. sha256Hex now falls back to a plain-JS SHA-256 (FIPS 180-4 vectors tested,
  including the padding boundary and a million bytes), so the SHA-256 check works everywhere.
- ReleaseRecordDetail: the 'Break-glass sessions during this release' panel hugged the columns above and had no colour
  strip; now the same 18 px gap as Human context and a strip in the break-glass red.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@jfillman
jfillman force-pushed the breakglass-ui-fixes branch from 940655f to c5d0176 Compare October 12, 2026 04:57
@jfillman
jfillman merged commit 76d1606 into main Oct 12, 2026
1 check passed
@jfillman
jfillman deleted the breakglass-ui-fixes branch October 12, 2026 05:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant