fix: update security-audit.html with 2026-08-17 audit results - #51
Merged
Conversation
Reconciliation pass: checked all 8 findings still marked "Open" in
the register against their linked GitHub issue's live state — all 8
(#782, #783, #784, #785, #786, #788, #789, #627) are now Closed.
Each row's status flipped to Closed with a "Resolved <date>" note
appended using the ticket's real closedAt date; finding text and
severity left untouched per the page's existing convention.
Adds a third findings batch ("audited 2026-08-17, commit 1104473"):
a Medium/Portability finding on traverse-runtime's ungated inference
module (#1065) and a Medium/Security finding on the gRPC EventService
lacking a concurrency cap (#1066, CWE-400).
#1065 is recorded Closed rather than Open — it was fixed and closed
today (2026-08-17, stateReason COMPLETED), same day as this audit.
The page explicitly documents that finding status reflects the
linked issue's live tracker state, not a re-audit snapshot, so this
keeps the page consistent with its own stated policy.
Also adds a "Security strengths" bullet noting the app-event/gRPC
eventing subsystem (auth, emit_event ABI, cross-workspace isolation)
was reviewed and found soundly designed — no new finding, since
nothing was wrong. Header date/commit and computed summary counts
updated; allFindings now includes the new batch so the executive
summary can't drift from the register.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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.
Summary
Reconciliation (Step 1): all 8 findings still marked "Open" in the register — #782, #783, #784, #785, #786, #788, #789, #627 — were checked against their linked GitHub issue's live state. All 8 are now Closed. Each row's status flipped to Closed with a "Resolved
<date>" note appended using the ticket's realclosedAtdate. No finding text, severity, or ordering was reworded — only status/date fields changed, per instructions.New findings (Step 2): added a third batch, "New findings — audited 2026-08-17, commit 1104473":
inferencemodule has no wasm32 gating despite blockingTcpStreamI/O (#1065)gh issue view— it was actually closed today, 2026-08-17,stateReason: COMPLETED, same day as this audit. The page's own findings-register note already states "Status reflects the linked GitHub issue's tracker state, not a re-audit of the fix," so I recorded it Closed to keep the page consistent with its own documented policy rather than publish a status I could see was already stale. #1066 is correctly Open as specified. Happy to flip #1065 back to Open if you'd rather the page reflect audit-day state specifically — just say so.Step 3: header updated to "Updated 2026-08-17, commit 1104473" (text + commit link href);
jsonLd.dateModifiedupdated to match. Summary counts are computed automatically from the findings arrays (not hand-typed), so they update correctly from both the reconciliation and the two new rows.Step 4: added one bullet to "Security strengths" noting the app-event/gRPC eventing subsystem was reviewed 2026-08-17 and found soundly designed — no new finding added, since nothing was wrong there.
Verified real before writing:
gh issue viewfor all 8 reconciled tickets + #1065/#1066, andgh api compareconfirming commit1104473is a real ancestor ofmain.Test plan
npm run build— 87 pages, no errorsstatus-pillon the whole page still reads "Open" (#1066)