Repository navigation
fix: stop retrying refused OpenChamber sign-ins; patch dependency advisories - #36
Merged
Merged
Conversation
Raise @modelcontextprotocol/core and /server to ^2.4.0 and zod to ^4.6.6 so fresh installs of the package (which ignore the lockfile) get the patched versions. Refresh dev dependencies; npm audit is clean.
A wrong password was posted again on every tool call and client retry, which reaches OpenChamber's failure limit (10 failures less than 5 minutes apart) and locks out the client IP, UI included, for 15 minutes. Hold a rejected password until restart, honour Retry-After on 429, hold other refusals for 60s, and keep retrying network errors and 5xx. Document the OpenChamber 1.12-1.24 requirement for 3.x and when to set a stable OPENCODE_OPENCHAMBER_CLIENT_ID.
This was referenced Oct 29, 2026
Merged
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.
Follow-ups to #34 before 3.1.0.
Changes
src/auth.ts). With a wrongOPENCODE_OPENCHAMBER_PASSWORD, every tool call and client retry posted the password again. OpenChamber locks the client IP out for 15 minutes, UI included, after 10 failures less than 5 minutes apart (checked in@openchamber/web1.24.0,ui-auth.js). Even a 60s cooldown would only delay the lockout. Now:Retry-After, in seconds or as an HTTP date;package.json).@modelcontextprotocol/coreand/servergo to^2.4.0, andzodto^4.6.6, for GHSA-jm63-cq47-j5jp, GHSA-46jr-4wx8-v453, GHSA-f897-62hf-6gxq, GHSA-mrhx-c7xc-m3w5 and GHSA-rvq5-wwqv-78pq. The ranges are raised, not only the lockfile, because the published package doesn't ship one. Dev dependencies are refreshed too.OPENCODE_OPENCHAMBER_CLIENT_IDso restarts don't pile up trusted-device entries.Validation
npm test: 525 passed, 3 skipped. There are new tests for a wrong password held until restart,Retry-After, 5xx and network-error retries, andRetry-Afterparsing. I confirmed the retry tests fail if every error is cached.tsc --noEmit,npm run docs:check,npm pack --dry-runandgit diff --checkpass, andnpm auditreports 0 vulnerabilities.