chore: never let publisher signing seeds into git - #9
Draft
lr00rl wants to merge 1 commit into
Draft
Conversation
cmd/pluginsign reads a raw 32-byte Ed25519 seed from a file path. Nothing is committed today, but the only thing between a publisher seed and main was one `git add -A` — so ignore key material outright rather than relying on care. A leaked publisher seed signs manifests the server loads as trusted, so the blast radius is the whole plugin trust model.
Contributor
Author
|
[ack] — zeus review, Olympus TASK-0007 seeds batch (2026-07-26): preventive guard verified — I independently re-scanned all-refs history ( |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is the repo where
cmd/pluginsignactually runs — it reads a raw 32-byte Ed25519 publisher seed from a file path (-seed seed.bin) — and.gitignoreexcluded none of*.key,*.pem,seed*,*.bin.Nothing is committed today: I scanned every tracked file and the full history (
--diff-filter=Aacross all refs) in all seven Lattice repos and found no key material. This is preventive — the only thing between a publisher seed andmainwas onegit add -A.A leaked publisher seed lets an attacker sign a plugin manifest the server loads as trusted, so the blast radius is the whole plugin trust model. Companion PRs add the same guard to the six plugin/index repos.
Found during the 2026-07-14 plugin-boundary architecture review (§4.8).
https://claude.ai/code/session_01FzYExyy6G7Cb7QpXoUEY78