Repository navigation
fix(security): harden account linking and host isolation - #72
Merged
Merged
Conversation
Bumps [@modelcontextprotocol/sdk](https://github.com/modelcontextprotocol/typescript-sdk) from 1.30.0 to 1.32.1. - [Release notes](https://github.com/modelcontextprotocol/typescript-sdk/releases) - [Commits](modelcontextprotocol/typescript-sdk@1.30.0...1.32.1) --- updated-dependencies: - dependency-name: "@modelcontextprotocol/sdk" dependency-version: 1.32.1 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
This was referenced Oct 14, 2026
…textprotocol/sdk-1.32.1' into HEAD
# Conflicts: # packages/client/src/cloud-relay.test.ts # skills/vectis/SKILL.md
# Conflicts: # packages/client/src/cloud-relay.test.ts
# Conflicts: # packages/client/src/remote-operation.ts
# Conflicts: # apps/server/src/service.ts # skills/vectis/SKILL.md # tests/convex/runnerLeases.test.ts
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.
The Codex Security Cloud report identified seven weaknesses in account linking, runner authorization, host isolation and site deployment. This PR addresses all seven findings.
csf_47282db2fad9a2f4d44249ee): bind OAuth initiation and callback to a server-generated Secure/HttpOnly browser cookie. Confirmation requires the original authenticated owner and a separate completion cookie. PKCE, expiry and replay protection remain enforced.csf_0851e5f73ece87c4e56acd62): require private repository fork workflows to be disabled or require approval, both when connecting a repository and before JIT registration. Preserve public repository checks, read-only queries and runner cleanup. Policy failures carry actionable guidance through the service and remote client.csf_aa32b876410df5e58be0585f): repair existing home permissions to 0700 and connection, SQLite and sidecar permissions to 0600 without losing state. Reject symlinks, non-regular files and foreign ownership, use umask 0077, and publish the connection atomically.csf_ec8096b288874408c19107dc): pin archive and font file sizes and hashes, extract only selected member bytes, and verify cached files without following links. Separate site build and deployment into disposable jobs; deployment checks the build's artifact digest and publishes the existing worker without rebuilding.csf_7cdbdd7278cf85f27f816b99): generate and pin a different host key for each instance, delivered through read-only identity media and installed before SSH starts. Refuse legacy SSH images, preserve their metadata, and expose the new requirement in manual desktop registration. The identity installer runs before SSH socket activation without creating a systemd boot dependency cycle.csf_a6a871f489b1ced3ac42b936): add a targeted QEMU/libslirp ingress filter that blocks host interface addresses, loopback, private and link-local destinations while preserving public IPv4 internet, virtual DNS, DHCP and the loopback SSH forward. Require a compatible patched runtime before VM or disk creation, including packaging and source verification.csf_949222f4c1766062fe4343b3): move the twopull_request_targetmetadata jobs to GitHub-hosted runners, because these events do not wait for fork approval before consuming Vectis capacity. Their checking and labeling behavior is unchanged.Ship the web and Convex changes together. Existing Linux SSH runner images must be prepared again. Windows requires the newly patched runtime and uses IPv4 guest networking; its existing base image does not need reinstalling. Documentation changes cover only these changed setup, approval and release requirements.
Validation:
pnpm checkpasses: 407 tests across 98 files, plus format, lint and typecheck. Headless and web/docs builds, artifact verification and the exact site deployment dry-run pass.Windows is not installed on the test host, so no Windows guest boot was exercised. The browser test uses synthetic providers; no live account linking, deployment or release was performed.
All Apple helpers build locally in release mode. The original Native CI job stopped before compilation because the Vectis macOS image lacks Xcode/Command Line Tools. The helper check temporarily uses GitHub's macOS 26 ARM64 runner, with the same build command, until that image has its toolchain. The original Ubuntu CI runner lost communication before uploading any step logs. On commit
98b7e2b, the full CI check passes on Vectis, the Native release build passes on the hosted ARM64 runner, and both PR metadata checks pass.