Conversation
… rig (Spec 107 PR-B)
… binding and refusals (Spec 107 PR-B)
…ns, groups capture and subject binding (Spec 107 PR-B) Related #1169
…e security and forced MCP auth (Spec 107 PR-B)
…ed proxies, cookie security policy, forced MCP auth, provider probe (Spec 107 PR-B)
… keys; telemetry v13 server-edition fields (Spec 107 PR-B)
T062/T063: real-instance verification of the OIDC front door (happy-path login, groups, /auth/me, agent-token mint + /mcp initialize/tools-list, ErrorMode tamper matrix, cookie Secure matrix behind a simulated trusted vs untrusted proxy) and the telemetry v13 payload (server vs personal). T064: full gate set re-run on HEAD after cross-review round 6 touched the config load path, oauth_handler.go and telemetry.go — all green. T066: tick completed PR-B tasks (T032-T066); ROADMAP.md regenerated. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Deploying mcpproxy-docs with
|
| Latest commit: |
2fdb86b
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://f31deabb.mcpproxy-docs.pages.dev |
| Branch Preview URL: | https://107-b-oidc-front-door.mcpproxy-docs.pages.dev |
Contributor
📦 Build ArtifactsWorkflow Run: View Run Available Artifacts
How to DownloadOption 1: GitHub Web UI (easiest)
Option 2: GitHub CLI gh run download 35168963326 --repo smart-mcp-proxy/mcpproxy-go
|
|
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
…1288) This branch diverged from main before #1288 landed, so PR-B's CI hit the exact regression it fixed: handleAddFromRegistry's success path dereferenced a nil cfg, chi's recoverer turned the panic into a bare 500, and the recovered hardware fault corrupted the Go heap on windows/amd64 under Go 1.26 (golang/go#81238) — the internal/httpapi test binary crashed with "found pointer to free object" and took the whole Windows unit-test job (and the binaries job's httpapi/scanner test bundle) down with it. Cherry-picked the same fix main already carries: the nil-config guard with a JSON 500 (and its pinning test), plus the unrelated Pass-2 temp-dir teardown race fix in the scanner test this same upstream commit bundled. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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
2 participants
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
Stacked on #1287 — merge that first; the base retargets to
mainautomatically.Spec 107 PR-B (Phase B.1–B.6, T032–T066): generic OIDC provider for the server edition's front door, running behind an ingress/reverse proxy, plus telemetry v13.
internal/serveredition/auth/{oidc_provider.go,oidc_jwks.go}, 142 lines against the 150-line budget): stdlib JWK→key parsing (crypto/rsa,crypto/ecdsa), per-kidcache, discovery, PKCE + nonce login, RS256id_tokenverification (iss/aud/azp/exp/iat/nonce),/userinfofallback, and the fullErrorModefail-closed matrix (bad signature, wrong issuer/audience, missing nonce,alg:none/HS256 confusion, malformed groups claim, discovery/token-endpoint failures) — every case refuses the login with no session issued, distinguishing the "closed" 403 class from the "unavailable" 503 class (FR-024).trusted_proxiesconfig gates whetherX-Forwarded-Proto/X-Real-IP/X-Forwarded-Hostare believed;session_cookie_secure: autoderives the cookie'sSecureattribute frompublic_url, in-process TLS, or a trusted proxy's forwarded scheme — an untrusted peer cannot spoofhttpsto change the decision.IssuerURL,AllowInsecureIssuer,Scopes,GroupsClaim,EmailVerifiedPolicy,DisplayNameonServerEditionOAuthConfig;PublicURL/SessionCookieSecureonServerEditionConfig; build-tagged accessors; publicGET /api/v1/auth/providerprobe so the Web UI learns the edition before authenticating; config-load path fixed (round 6) so a resolved OAuth client secret is never round-tripped back intomcp_config.jsonby an unrelatedPATCH /api/v1/config.server_edition_enabled,idp_provider,member_count_bucket(deliberately notuser_count_bucket— see the rejected finding in cross-review round 6: auser-prefixed key would trip the PII scanner's home-dir-basename check on everyUSER user-topology server container) added to the heartbeat payload; personal builds reportserver_edition_enabled=false,idp_provider="none".scripts/dev-server-edition.sh: local verification rig implementing quickstart §0–§4 and §6 (--phase b|c|d), used for both the fixed test suite and the real-instance verification below.Key config
trusted_proxies[]string(top-level, edition-neutral)X-Forwarded-*server_edition.public_urlhttpsscheme forcesSecurecookies unconditionallyserver_edition.session_cookie_secure"auto"|"true"|"false"autoserver_edition.oauth.provider"oidc"(new; alongside existing)server_edition.oauth.issuer_urlhttpsunlessallow_insecure_issuer(loopback dev only)server_edition.oauth.groups_claim,.email_verified_policy,.display_nameVerification gates (short)
All green on HEAD
873145b64(re-run in full after cross-review round 6 touched the config load path,oauth_handler.go, andtelemetry.go): both editions build;go vetboth tags;golangci-lintv2.9.0 both tag sets (0 issues); server-edition + personal race suites (incl. theinternal/server/internal/httpapiCI-skip sweep); frozen tool-surface goldens;go test ./cmd/...,./tests/oauthserver/...incl.OAUTH_INTEGRATION_TESTS=1;make swagger-verify+TestContractsInSync;gen-roadmap.py --check; frontendvitest(1284 tests) +npm run build;gofmt -lclean;check-settings-parity.py(61 settings consistent); Swiftswift test(1166/1167 — the one failure is the same pre-existing environmental red herring as PR-A,AppLifecycleTestscolliding with a live local tray journal); isolatedscripts/test-api-e2e.sh(65/65). Full command-by-command log:specs/107-server-edition-sso-hardening/verification.md→## PR-B→### Automated checks.Real-instance verification (
### Real instancein the same file): headless OIDC login end-to-end against a fake IdP (tests/oauthserver), groups on/auth/me, agent-token mint +/mcpinitialize/tools/list→ 200, the fullErrorModetamper matrix, and the cookie-Secure matrix behind a simulated trusted-vs-untrusted reverse proxy. Two noted discrepancies between the task text and the shipped code (task rule: code wins) are called out inline: quickstart §6 is reachable only via--phase d, which needs PR-C's entitlement filtering (§5) first — verified directly with curl instead; and the telemetry wire key ismember_count_bucket, notuser_count_bucket.6 rounds of cross-model review (
opencode run --model github-copilot/gpt-6-astra, fallbackgpt-5.6-sol), all findings verified against the live tree before fixing; details inverification.md→### Cross-review.Related #1169 (live-role reuse).
🤖 Generated with Claude Code