hosts + HD agent: in-band auth, ownership-checked pipeline, verified agent download, Linux restart, CI fixes - #21
Merged
Merged
Conversation
…ified agent download, Linux restart on upgrade, CI fixes
hiperf-agent.py: Exp-Golomb reader fixed (suffix from 0 plus offset), so
non-IDR I-slices classify correctly. Auth accepts a first-frame
{"type":"auth","token"} within 5 s; ?token= still works but logs a
deprecation. Every await-then-mutate in start/stop/_spawn_pipeline checks
(ws, gen) ownership; a stale spawn kills what it created. 19 new tests in
computer-viewer/tests/test-hiperf-agent.py.
hiperf-mac/linux/windows: the downloaded hiperf-agent.py is staged and its
SHA-256 checked against MANIFEST.sha256 at the same ref (local checkout
manifest as offline fallback) before it replaces the installed agent.
connect-linux/hiperf-linux: enable, then restart if active else start, and
print the effective listener; enable --now never restarted a live unit so
upgrades kept the old bind. test-bind.sh covers it with shimmed systemctl.
connect-windows.ps1: TightVNC MSI and UltraVNC zip pinned by SHA-256
(Get-FileHash before use); Amyuni's unversioned zip is gated on the
WHQL-signed catalog's Authenticode signature (the exe itself is unsigned).
PowerShell edits are unexecuted here; CI now parses all four .ps1 files on
windows-latest.
CI + release.sh: xargs -0 -n 1 bash -n (the batched form only parsed the
first file); the HD agent tests run on all OSes. install.sh also installs
hiperf-linux.sh and the two Windows scripts. release.sh accepts a same-day
follow-up tag vYYYY.MM.DD.N. Manifest regenerated.
Co-Authored-By: Claude Fable 5.1 <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
1 participant
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.
Round-2 review fixes (Codex 2, 4, 5, 6, 14; Grok 6, 13; Claude 7, 8, 9, 12 server side). Also lets release.sh cut a same-day follow-up tag. Full suite green: 49 JS, 19 HD-agent, 27 orgo, 7 texting-style, installer/bind/agent-installer scripts, manifest check.
🤖 Generated with Claude Code