skybridge 0.0.0-dev.ecfdeac → 0.0.0-dev.ed29eee
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +2 -9
- package/dist/cli/build-helpers.d.ts +6 -2
- package/dist/cli/build-helpers.js +37 -12
- package/dist/cli/build-helpers.js.map +1 -1
- package/dist/cli/build-helpers.test.js +71 -3
- package/dist/cli/build-helpers.test.js.map +1 -1
- package/dist/cli/build-steps.d.ts +2 -0
- package/dist/cli/build-steps.js +73 -0
- package/dist/cli/build-steps.js.map +1 -0
- package/dist/cli/build-steps.test.d.ts +1 -0
- package/dist/cli/build-steps.test.js +52 -0
- package/dist/cli/build-steps.test.js.map +1 -0
- package/dist/cli/header.d.ts +1 -1
- package/dist/cli/resolve-skybridge-config.d.ts +5 -0
- package/dist/cli/{resolve-views-dir.js → resolve-skybridge-config.js} +4 -3
- package/dist/cli/resolve-skybridge-config.js.map +1 -0
- package/dist/cli/run-plain.d.ts +18 -0
- package/dist/cli/run-plain.js +89 -0
- package/dist/cli/run-plain.js.map +1 -0
- package/dist/cli/use-nodemon.d.ts +14 -0
- package/dist/cli/use-nodemon.js +71 -60
- package/dist/cli/use-nodemon.js.map +1 -1
- package/dist/cli/use-typescript-check.d.ts +8 -2
- package/dist/cli/use-typescript-check.js +70 -67
- package/dist/cli/use-typescript-check.js.map +1 -1
- package/dist/commands/build.d.ts +0 -2
- package/dist/commands/build.js +2 -61
- package/dist/commands/build.js.map +1 -1
- package/dist/commands/dev.d.ts +1 -0
- package/dist/commands/dev.js +34 -2
- package/dist/commands/dev.js.map +1 -1
- package/dist/context-warnings.d.ts +5 -0
- package/dist/context-warnings.js +31 -0
- package/dist/context-warnings.js.map +1 -0
- package/dist/context-warnings.test.d.ts +1 -0
- package/dist/context-warnings.test.js +28 -0
- package/dist/context-warnings.test.js.map +1 -0
- package/dist/server/auth/index.d.ts +30 -4
- package/dist/server/auth/index.js.map +1 -1
- package/dist/server/auth/providers/auth0.d.ts +24 -2
- package/dist/server/auth/providers/auth0.js.map +1 -1
- package/dist/server/auth/providers/auth0.test.js +4 -2
- package/dist/server/auth/providers/auth0.test.js.map +1 -1
- package/dist/server/auth/providers/authplane.d.ts +49 -0
- package/dist/server/auth/providers/authplane.js +57 -0
- package/dist/server/auth/providers/authplane.js.map +1 -0
- package/dist/server/auth/providers/authplane.test.d.ts +1 -0
- package/dist/server/auth/providers/authplane.test.js +140 -0
- package/dist/server/auth/providers/authplane.test.js.map +1 -0
- package/dist/server/auth/providers/clerk.d.ts +39 -2
- package/dist/server/auth/providers/clerk.js +4 -1
- package/dist/server/auth/providers/clerk.js.map +1 -1
- package/dist/server/auth/providers/clerk.test.js +5 -3
- package/dist/server/auth/providers/clerk.test.js.map +1 -1
- package/dist/server/auth/providers/custom.d.ts +18 -2
- package/dist/server/auth/providers/custom.js +29 -10
- package/dist/server/auth/providers/custom.js.map +1 -1
- package/dist/server/auth/providers/custom.test.js +61 -10
- package/dist/server/auth/providers/custom.test.js.map +1 -1
- package/dist/server/auth/providers/descope.d.ts +36 -5
- package/dist/server/auth/providers/descope.js +18 -5
- package/dist/server/auth/providers/descope.js.map +1 -1
- package/dist/server/auth/providers/descope.test.js +42 -17
- package/dist/server/auth/providers/descope.test.js.map +1 -1
- package/dist/server/auth/providers/stytch.d.ts +15 -2
- package/dist/server/auth/providers/stytch.js +4 -1
- package/dist/server/auth/providers/stytch.js.map +1 -1
- package/dist/server/auth/providers/verify-spy.d.ts +12 -0
- package/dist/server/auth/providers/verify-spy.js +21 -0
- package/dist/server/auth/providers/verify-spy.js.map +1 -0
- package/dist/server/auth/providers/workos.d.ts +24 -2
- package/dist/server/auth/providers/workos.js +4 -1
- package/dist/server/auth/providers/workos.js.map +1 -1
- package/dist/server/auth/security-schemes.d.ts +23 -0
- package/dist/server/auth/security-schemes.js +64 -0
- package/dist/server/auth/security-schemes.js.map +1 -0
- package/dist/server/auth/security-schemes.test.d.ts +1 -0
- package/dist/server/auth/security-schemes.test.js +99 -0
- package/dist/server/auth/security-schemes.test.js.map +1 -0
- package/dist/server/auth/setup.d.ts +3 -1
- package/dist/server/auth/setup.js +65 -24
- package/dist/server/auth/setup.js.map +1 -1
- package/dist/server/auth/setup.test.js +328 -5
- package/dist/server/auth/setup.test.js.map +1 -1
- package/dist/server/auth/verify.d.ts +37 -3
- package/dist/server/auth/verify.js +26 -2
- package/dist/server/auth/verify.js.map +1 -1
- package/dist/server/auth/verify.test.js +49 -0
- package/dist/server/auth/verify.test.js.map +1 -1
- package/dist/server/auth-extra.test-d.d.ts +1 -0
- package/dist/server/auth-extra.test-d.js +82 -0
- package/dist/server/auth-extra.test-d.js.map +1 -0
- package/dist/server/auth.d.ts +39 -1
- package/dist/server/auth.js.map +1 -1
- package/dist/server/host.d.ts +2 -0
- package/dist/server/host.js +11 -0
- package/dist/server/host.js.map +1 -0
- package/dist/server/host.test.d.ts +1 -0
- package/dist/server/host.test.js +13 -0
- package/dist/server/host.test.js.map +1 -0
- package/dist/server/index.d.ts +11 -7
- package/dist/server/index.js +6 -2
- package/dist/server/index.js.map +1 -1
- package/dist/server/middleware.d.ts +31 -6
- package/dist/server/middleware.js +33 -0
- package/dist/server/middleware.js.map +1 -1
- package/dist/server/middleware.test.js +37 -1
- package/dist/server/middleware.test.js.map +1 -1
- package/dist/server/server.d.ts +87 -25
- package/dist/server/server.js +221 -120
- package/dist/server/server.js.map +1 -1
- package/dist/server/skills-integration.test.d.ts +1 -0
- package/dist/server/skills-integration.test.js +79 -0
- package/dist/server/skills-integration.test.js.map +1 -0
- package/dist/server/skills.d.ts +29 -0
- package/dist/server/skills.js +195 -0
- package/dist/server/skills.js.map +1 -0
- package/dist/server/skills.test.d.ts +1 -0
- package/dist/server/skills.test.js +247 -0
- package/dist/server/skills.test.js.map +1 -0
- package/dist/server/view-name.test-d.d.ts +1 -0
- package/dist/server/view-name.test-d.js +8 -0
- package/dist/server/view-name.test-d.js.map +1 -0
- package/dist/server/view-resource-resolution.test.js +93 -10
- package/dist/server/view-resource-resolution.test.js.map +1 -1
- package/dist/server/viewsDevServer.js +4 -1
- package/dist/server/viewsDevServer.js.map +1 -1
- package/dist/test/utils.d.ts +3 -3
- package/dist/test/view.test.js +86 -102
- package/dist/test/view.test.js.map +1 -1
- package/dist/web/bridges/adaptor.d.ts +51 -0
- package/dist/web/bridges/adaptor.js +327 -0
- package/dist/web/bridges/adaptor.js.map +1 -0
- package/dist/web/bridges/adaptor.test.d.ts +1 -0
- package/dist/web/bridges/adaptor.test.js +225 -0
- package/dist/web/bridges/adaptor.test.js.map +1 -0
- package/dist/web/bridges/apps-sdk/bridge.d.ts +6 -2
- package/dist/web/bridges/apps-sdk/bridge.js +15 -4
- package/dist/web/bridges/apps-sdk/bridge.js.map +1 -1
- package/dist/web/bridges/apps-sdk/index.d.ts +0 -1
- package/dist/web/bridges/apps-sdk/index.js +0 -1
- package/dist/web/bridges/apps-sdk/index.js.map +1 -1
- package/dist/web/bridges/get-adaptor.d.ts +4 -4
- package/dist/web/bridges/get-adaptor.js +6 -11
- package/dist/web/bridges/get-adaptor.js.map +1 -1
- package/dist/web/bridges/get-adaptor.test.d.ts +1 -0
- package/dist/web/bridges/get-adaptor.test.js +32 -0
- package/dist/web/bridges/get-adaptor.test.js.map +1 -0
- package/dist/web/bridges/mcp-app/bridge.d.ts +13 -2
- package/dist/web/bridges/mcp-app/bridge.js +51 -7
- package/dist/web/bridges/mcp-app/bridge.js.map +1 -1
- package/dist/web/bridges/mcp-app/bridge.test.d.ts +1 -0
- package/dist/web/bridges/mcp-app/bridge.test.js +17 -0
- package/dist/web/bridges/mcp-app/bridge.test.js.map +1 -0
- package/dist/web/bridges/mcp-app/index.d.ts +0 -1
- package/dist/web/bridges/mcp-app/index.js +0 -1
- package/dist/web/bridges/mcp-app/index.js.map +1 -1
- package/dist/web/bridges/mcp-app/types.d.ts +2 -0
- package/dist/web/bridges/mcp-app/types.js.map +1 -1
- package/dist/web/bridges/mcp-app/view-tools.test.js +5 -6
- package/dist/web/bridges/mcp-app/view-tools.test.js.map +1 -1
- package/dist/web/bridges/types.d.ts +10 -0
- package/dist/web/bridges/types.js +14 -1
- package/dist/web/bridges/types.js.map +1 -1
- package/dist/web/bridges/types.test.d.ts +1 -0
- package/dist/web/bridges/types.test.js +19 -0
- package/dist/web/bridges/types.test.js.map +1 -0
- package/dist/web/components/modal-provider.d.ts +1 -1
- package/dist/web/components/modal-provider.js +4 -3
- package/dist/web/components/modal-provider.js.map +1 -1
- package/dist/web/create-store.test.js +8 -5
- package/dist/web/create-store.test.js.map +1 -1
- package/dist/web/data-llm.d.ts +1 -1
- package/dist/web/data-llm.test.js +18 -5
- package/dist/web/data-llm.test.js.map +1 -1
- package/dist/web/hooks/index.d.ts +1 -0
- package/dist/web/hooks/index.js +1 -0
- package/dist/web/hooks/index.js.map +1 -1
- package/dist/web/hooks/test/utils.d.ts +2 -1
- package/dist/web/hooks/test/utils.js +1 -1
- package/dist/web/hooks/test/utils.js.map +1 -1
- package/dist/web/hooks/use-call-tool.test.js +37 -23
- package/dist/web/hooks/use-call-tool.test.js.map +1 -1
- package/dist/web/hooks/use-display-mode.test.js +56 -20
- package/dist/web/hooks/use-display-mode.test.js.map +1 -1
- package/dist/web/hooks/use-download.test.js +11 -3
- package/dist/web/hooks/use-download.test.js.map +1 -1
- package/dist/web/hooks/use-files.test.js +10 -2
- package/dist/web/hooks/use-files.test.js.map +1 -1
- package/dist/web/hooks/use-host-info.d.ts +22 -0
- package/dist/web/hooks/use-host-info.js +30 -0
- package/dist/web/hooks/use-host-info.js.map +1 -0
- package/dist/web/hooks/use-host-info.test.d.ts +1 -0
- package/dist/web/hooks/use-host-info.test.js +59 -0
- package/dist/web/hooks/use-host-info.test.js.map +1 -0
- package/dist/web/hooks/use-layout.test.js +59 -26
- package/dist/web/hooks/use-layout.test.js.map +1 -1
- package/dist/web/hooks/use-open-external.test.js +13 -3
- package/dist/web/hooks/use-open-external.test.js.map +1 -1
- package/dist/web/hooks/use-request-close.test.js +35 -40
- package/dist/web/hooks/use-request-close.test.js.map +1 -1
- package/dist/web/hooks/use-request-modal.test.js +10 -0
- package/dist/web/hooks/use-request-modal.test.js.map +1 -1
- package/dist/web/hooks/use-request-size.test.js +35 -53
- package/dist/web/hooks/use-request-size.test.js.map +1 -1
- package/dist/web/hooks/use-set-open-in-app-url.test.js +28 -8
- package/dist/web/hooks/use-set-open-in-app-url.test.js.map +1 -1
- package/dist/web/hooks/use-tool-info.test.js +37 -27
- package/dist/web/hooks/use-tool-info.test.js.map +1 -1
- package/dist/web/hooks/use-user.test.js +81 -35
- package/dist/web/hooks/use-user.test.js.map +1 -1
- package/dist/web/hooks/use-view-state.js +19 -9
- package/dist/web/hooks/use-view-state.js.map +1 -1
- package/dist/web/hooks/use-view-state.test.js +37 -2
- package/dist/web/hooks/use-view-state.test.js.map +1 -1
- package/dist/web/plugin/plugin-build.test.d.ts +1 -0
- package/dist/web/plugin/plugin-build.test.js +54 -0
- package/dist/web/plugin/plugin-build.test.js.map +1 -0
- package/dist/web/plugin/plugin.d.ts +6 -0
- package/dist/web/plugin/plugin.js +18 -7
- package/dist/web/plugin/plugin.js.map +1 -1
- package/dist/web/plugin/plugin.test.d.ts +1 -0
- package/dist/web/plugin/plugin.test.js +61 -0
- package/dist/web/plugin/plugin.test.js.map +1 -0
- package/package.json +21 -20
- package/dist/cli/resolve-views-dir.d.ts +0 -1
- package/dist/cli/resolve-views-dir.js.map +0 -1
- package/dist/web/bridges/apps-sdk/adaptor.d.ts +0 -29
- package/dist/web/bridges/apps-sdk/adaptor.js +0 -117
- package/dist/web/bridges/apps-sdk/adaptor.js.map +0 -1
- package/dist/web/bridges/mcp-app/adaptor.d.ts +0 -53
- package/dist/web/bridges/mcp-app/adaptor.js +0 -283
- package/dist/web/bridges/mcp-app/adaptor.js.map +0 -1
|
@@ -1,14 +1,22 @@
|
|
|
1
1
|
import { getOAuthProtectedResourceMetadataUrl, mcpAuthMetadataRouter, } from "@modelcontextprotocol/sdk/server/auth/router.js";
|
|
2
2
|
import cors from "cors";
|
|
3
|
-
import { requireBearerAuth } from "../auth.js";
|
|
3
|
+
import { optionalBearerAuth, requireBearerAuth } from "../auth.js";
|
|
4
4
|
import { resolveServerOrigin } from "../requestOrigin.js";
|
|
5
|
+
import { clientPrefersInBandChallenge, evaluateSecuritySchemes, httpStatusForFailure, inBandChallengeResult, securitySchemesAllowAnonymous, wwwAuthenticateHeader, } from "./security-schemes.js";
|
|
5
6
|
import { createJwksVerifier } from "./verify.js";
|
|
6
7
|
/** Mounts the well-known OAuth metadata and bearer auth on `/mcp`. */
|
|
7
|
-
export function setupOAuth(app, config) {
|
|
8
|
-
if (!config.verify
|
|
9
|
-
throw new Error("oauth
|
|
8
|
+
export function setupOAuth(app, config, schemesByTool) {
|
|
9
|
+
if (!config.verifier && !config.verify) {
|
|
10
|
+
throw new Error("oauth requires a `verifier`");
|
|
10
11
|
}
|
|
11
|
-
const
|
|
12
|
+
const acceptsAnonymous = () => [...schemesByTool.values()].some(securitySchemesAllowAnonymous);
|
|
13
|
+
const verifier = config.verifier ?? createJwksVerifier(config.verify);
|
|
14
|
+
const bearer = (options) => {
|
|
15
|
+
const required = requireBearerAuth(options);
|
|
16
|
+
const optional = optionalBearerAuth(options);
|
|
17
|
+
return (req, res, next) => (acceptsAnonymous() ? optional : required)(req, res, next);
|
|
18
|
+
};
|
|
19
|
+
let resourceMetadataUrl;
|
|
12
20
|
// baseUrl known at boot: bake the resource URLs once, no Host-header trust.
|
|
13
21
|
if (config.baseUrl !== undefined) {
|
|
14
22
|
let baseUrl;
|
|
@@ -23,29 +31,62 @@ export function setupOAuth(app, config) {
|
|
|
23
31
|
resourceServerUrl: baseUrl,
|
|
24
32
|
scopesSupported: config.scopesSupported,
|
|
25
33
|
}));
|
|
26
|
-
|
|
27
|
-
|
|
28
|
-
requiredScopes: config.requiredScopes,
|
|
29
|
-
resourceMetadataUrl: getOAuthProtectedResourceMetadataUrl(baseUrl),
|
|
30
|
-
}));
|
|
31
|
-
return;
|
|
34
|
+
const url = getOAuthProtectedResourceMetadataUrl(baseUrl);
|
|
35
|
+
resourceMetadataUrl = () => url;
|
|
32
36
|
}
|
|
33
|
-
|
|
34
|
-
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
|
|
42
|
-
|
|
37
|
+
else {
|
|
38
|
+
// No baseUrl: resolve the resource origin per request from forwarded headers
|
|
39
|
+
// (same precedence the framework uses for view serverUrl). Origin headers are
|
|
40
|
+
// pathless, so the PRM path stays at root, matching the SDK's static layout.
|
|
41
|
+
const resolveOrigin = (getHeader) => new URL(resolveServerOrigin(getHeader));
|
|
42
|
+
app.use("/.well-known/oauth-authorization-server", cors(), (_req, res) => void res.json(config.oauthMetadata));
|
|
43
|
+
app.use("/.well-known/oauth-protected-resource", cors(), (req, res) => {
|
|
44
|
+
res.json({
|
|
45
|
+
resource: resolveOrigin((key) => req.get(key)).href,
|
|
46
|
+
authorization_servers: [config.oauthMetadata.issuer],
|
|
47
|
+
scopes_supported: config.scopesSupported,
|
|
48
|
+
});
|
|
43
49
|
});
|
|
44
|
-
|
|
45
|
-
|
|
50
|
+
resourceMetadataUrl = (getHeader) => getOAuthProtectedResourceMetadataUrl(resolveOrigin(getHeader));
|
|
51
|
+
}
|
|
52
|
+
app.use("/mcp", (req, res, next) => bearer({
|
|
46
53
|
verifier,
|
|
47
54
|
requiredScopes: config.requiredScopes,
|
|
48
|
-
resourceMetadataUrl:
|
|
55
|
+
resourceMetadataUrl: resourceMetadataUrl((key) => req.get(key)),
|
|
49
56
|
})(req, res, next));
|
|
57
|
+
app.use("/mcp", (req, res, next) => {
|
|
58
|
+
const body = req.body;
|
|
59
|
+
const calls = Array.isArray(body) ? body : [body];
|
|
60
|
+
const auth = req.auth;
|
|
61
|
+
let failure;
|
|
62
|
+
for (const call of calls) {
|
|
63
|
+
const tool = call?.params?.name;
|
|
64
|
+
if (call?.method === "tools/call" && tool && schemesByTool.has(tool)) {
|
|
65
|
+
failure = evaluateSecuritySchemes(schemesByTool.get(tool), auth);
|
|
66
|
+
if (failure) {
|
|
67
|
+
break;
|
|
68
|
+
}
|
|
69
|
+
}
|
|
70
|
+
}
|
|
71
|
+
if (!failure) {
|
|
72
|
+
return next();
|
|
73
|
+
}
|
|
74
|
+
const url = resourceMetadataUrl((key) => req.get(key));
|
|
75
|
+
if (!Array.isArray(body) &&
|
|
76
|
+
clientPrefersInBandChallenge(req.get("user-agent"))) {
|
|
77
|
+
res.json({
|
|
78
|
+
jsonrpc: "2.0",
|
|
79
|
+
id: body?.id ?? null,
|
|
80
|
+
result: inBandChallengeResult(failure, url),
|
|
81
|
+
});
|
|
82
|
+
return;
|
|
83
|
+
}
|
|
84
|
+
res.set("WWW-Authenticate", wwwAuthenticateHeader(failure, url));
|
|
85
|
+
res.status(httpStatusForFailure(failure)).json({
|
|
86
|
+
error: failure.error,
|
|
87
|
+
error_description: failure.description,
|
|
88
|
+
});
|
|
89
|
+
});
|
|
90
|
+
return resourceMetadataUrl;
|
|
50
91
|
}
|
|
51
92
|
//# sourceMappingURL=setup.js.map
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"setup.js","sourceRoot":"","sources":["../../../src/server/auth/setup.ts"],"names":[],"mappings":"
|
|
1
|
+
{"version":3,"file":"setup.js","sourceRoot":"","sources":["../../../src/server/auth/setup.ts"],"names":[],"mappings":"AACA,OAAO,EACL,oCAAoC,EACpC,qBAAqB,GACtB,MAAM,iDAAiD,CAAC;AACzD,OAAO,IAAI,MAAM,MAAM,CAAC;AAGxB,OAAO,EAAE,kBAAkB,EAAE,iBAAiB,EAAE,MAAM,YAAY,CAAC;AACnE,OAAO,EAAE,mBAAmB,EAAE,MAAM,qBAAqB,CAAC;AAG1D,OAAO,EACL,4BAA4B,EAC5B,uBAAuB,EACvB,oBAAoB,EACpB,qBAAqB,EAErB,6BAA6B,EAC7B,qBAAqB,GACtB,MAAM,uBAAuB,CAAC;AAC/B,OAAO,EAAE,kBAAkB,EAAE,MAAM,aAAa,CAAC;AAMjD,sEAAsE;AACtE,MAAM,UAAU,UAAU,CACxB,GAAY,EACZ,MAA4C,EAC5C,aAAwD;IAExD,IAAI,CAAC,MAAM,CAAC,QAAQ,IAAI,CAAC,MAAM,CAAC,MAAM,EAAE,CAAC;QACvC,MAAM,IAAI,KAAK,CAAC,6BAA6B,CAAC,CAAC;IACjD,CAAC;IAED,MAAM,gBAAgB,GAAG,GAAG,EAAE,CAC5B,CAAC,GAAG,aAAa,CAAC,MAAM,EAAE,CAAC,CAAC,IAAI,CAAC,6BAA6B,CAAC,CAAC;IAClE,MAAM,QAAQ,GAAG,MAAM,CAAC,QAAQ,IAAI,kBAAkB,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC;IACtE,MAAM,MAAM,GAAG,CAAC,OAAoC,EAAkB,EAAE;QACtE,MAAM,QAAQ,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAC;QAC5C,MAAM,QAAQ,GAAG,kBAAkB,CAAC,OAAO,CAAC,CAAC;QAC7C,OAAO,CAAC,GAAG,EAAE,GAAG,EAAE,IAAI,EAAE,EAAE,CACxB,CAAC,gBAAgB,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,GAAG,EAAE,GAAG,EAAE,IAAI,CAAC,CAAC;IAC/D,CAAC,CAAC;IAEF,IAAI,mBAAgD,CAAC;IAErD,4EAA4E;IAC5E,IAAI,MAAM,CAAC,OAAO,KAAK,SAAS,EAAE,CAAC;QACjC,IAAI,OAAY,CAAC;QACjB,IAAI,CAAC;YACH,OAAO,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;QACpC,CAAC;QAAC,MAAM,CAAC;YACP,MAAM,IAAI,KAAK,CACb,oDAAoD,IAAI,CAAC,SAAS,CAChE,MAAM,CAAC,OAAO,CACf,EAAE,CACJ,CAAC;QACJ,CAAC;QACD,GAAG,CAAC,GAAG,CACL,qBAAqB,CAAC;YACpB,aAAa,EAAE,MAAM,CAAC,aAAa;YACnC,iBAAiB,EAAE,OAAO;YAC1B,eAAe,EAAE,MAAM,CAAC,eAAe;SACxC,CAAC,CACH,CAAC;QACF,MAAM,GAAG,GAAG,oCAAoC,CAAC,OAAO,CAAC,CAAC;QAC1D,mBAAmB,GAAG,GAAG,EAAE,CAAC,GAAG,CAAC;IAClC,CAAC;SAAM,CAAC;QACN,6EAA6E;QAC7E,8EAA8E;QAC9E,6EAA6E;QAC7E,MAAM,aAAa,GAAG,CAAC,SAA8C,EAAE,EAAE,CACvE,IAAI,GAAG,CAAC,mBAAmB,CAAC,SAAS,CAAC,CAAC,CAAC;QAC1C,GAAG,CAAC,GAAG,CACL,yCAAyC,EACzC,IAAI,EAAE,EACN,CAAC,IAAI,EAAE,GAAG,EAAE,EAAE,CAAC,KAAK,GAAG,CAAC,IAAI,CAAC,MAAM,CAAC,aAAa,CAAC,CACnD,CAAC;QACF,GAAG,CAAC,GAAG,CAAC,uCAAuC,EAAE,IAAI,EAAE,EAAE,CAAC,GAAG,EAAE,GAAG,EAAE,EAAE;YACpE,GAAG,CAAC,IAAI,CAAC;gBACP,QAAQ,EAAE,aAAa,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI;gBACnD,qBAAqB,EAAE,CAAC,MAAM,CAAC,aAAa,CAAC,MAAM,CAAC;gBACpD,gBAAgB,EAAE,MAAM,CAAC,eAAe;aACzC,CAAC,CAAC;QACL,CAAC,CAAC,CAAC;QACH,mBAAmB,GAAG,CAAC,SAAS,EAAE,EAAE,CAClC,oCAAoC,CAAC,aAAa,CAAC,SAAS,CAAC,CAAC,CAAC;IACnE,CAAC;IAED,GAAG,CAAC,GAAG,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,GAAG,EAAE,IAAI,EAAE,EAAE,CACjC,MAAM,CAAC;QACL,QAAQ;QACR,cAAc,EAAE,MAAM,CAAC,cAAc;QACrC,mBAAmB,EAAE,mBAAmB,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;KAChE,CAAC,CAAC,GAAG,EAAE,GAAG,EAAE,IAAI,CAAC,CACnB,CAAC;IAEF,GAAG,CAAC,GAAG,CAAC,MAAM,EAAE,CAAC,GAAG,EAAE,GAAG,EAAE,IAAI,EAAE,EAAE;QAMjC,MAAM,IAAI,GAAG,GAAG,CAAC,IAAiC,CAAC;QACnD,MAAM,KAAK,GAAG,KAAK,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QAClD,MAAM,IAAI,GAAI,GAAqC,CAAC,IAAI,CAAC;QACzD,IAAI,OAAkC,CAAC;QACvC,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;YACzB,MAAM,IAAI,GAAG,IAAI,EAAE,MAAM,EAAE,IAAI,CAAC;YAChC,IAAI,IAAI,EAAE,MAAM,KAAK,YAAY,IAAI,IAAI,IAAI,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC;gBACrE,OAAO,GAAG,uBAAuB,CAAC,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,IAAI,CAAC,CAAC;gBACjE,IAAI,OAAO,EAAE,CAAC;oBACZ,MAAM;gBACR,CAAC;YACH,CAAC;QACH,CAAC;QACD,IAAI,CAAC,OAAO,EAAE,CAAC;YACb,OAAO,IAAI,EAAE,CAAC;QAChB,CAAC;QACD,MAAM,GAAG,GAAG,mBAAmB,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,CAAC;QACvD,IACE,CAAC,KAAK,CAAC,OAAO,CAAC,IAAI,CAAC;YACpB,4BAA4B,CAAC,GAAG,CAAC,GAAG,CAAC,YAAY,CAAC,CAAC,EACnD,CAAC;YACD,GAAG,CAAC,IAAI,CAAC;gBACP,OAAO,EAAE,KAAK;gBACd,EAAE,EAAE,IAAI,EAAE,EAAE,IAAI,IAAI;gBACpB,MAAM,EAAE,qBAAqB,CAAC,OAAO,EAAE,GAAG,CAAC;aAC5C,CAAC,CAAC;YACH,OAAO;QACT,CAAC;QACD,GAAG,CAAC,GAAG,CAAC,kBAAkB,EAAE,qBAAqB,CAAC,OAAO,EAAE,GAAG,CAAC,CAAC,CAAC;QACjE,GAAG,CAAC,MAAM,CAAC,oBAAoB,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC;YAC7C,KAAK,EAAE,OAAO,CAAC,KAAK;YACpB,iBAAiB,EAAE,OAAO,CAAC,WAAW;SACvC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,OAAO,mBAAmB,CAAC;AAC7B,CAAC","sourcesContent":["import type { BearerAuthMiddlewareOptions } from \"@modelcontextprotocol/sdk/server/auth/middleware/bearerAuth.js\";\nimport {\n getOAuthProtectedResourceMetadataUrl,\n mcpAuthMetadataRouter,\n} from \"@modelcontextprotocol/sdk/server/auth/router.js\";\nimport cors from \"cors\";\nimport type { Express, Request, RequestHandler } from \"express\";\nimport type { AuthInfo } from \"../auth.js\";\nimport { optionalBearerAuth, requireBearerAuth } from \"../auth.js\";\nimport { resolveServerOrigin } from \"../requestOrigin.js\";\nimport type { SecurityScheme } from \"../server.js\";\nimport type { OAuthConfig } from \"./index.js\";\nimport {\n clientPrefersInBandChallenge,\n evaluateSecuritySchemes,\n httpStatusForFailure,\n inBandChallengeResult,\n type SchemeFailure,\n securitySchemesAllowAnonymous,\n wwwAuthenticateHeader,\n} from \"./security-schemes.js\";\nimport { createJwksVerifier } from \"./verify.js\";\n\nexport type ResourceMetadataUrlResolver = (\n getHeader: (key: string) => string | undefined,\n) => string;\n\n/** Mounts the well-known OAuth metadata and bearer auth on `/mcp`. */\nexport function setupOAuth(\n app: Express,\n config: OAuthConfig<Record<string, unknown>>,\n schemesByTool: Map<string, SecurityScheme[] | undefined>,\n): ResourceMetadataUrlResolver {\n if (!config.verifier && !config.verify) {\n throw new Error(\"oauth requires a `verifier`\");\n }\n\n const acceptsAnonymous = () =>\n [...schemesByTool.values()].some(securitySchemesAllowAnonymous);\n const verifier = config.verifier ?? createJwksVerifier(config.verify);\n const bearer = (options: BearerAuthMiddlewareOptions): RequestHandler => {\n const required = requireBearerAuth(options);\n const optional = optionalBearerAuth(options);\n return (req, res, next) =>\n (acceptsAnonymous() ? optional : required)(req, res, next);\n };\n\n let resourceMetadataUrl: ResourceMetadataUrlResolver;\n\n // baseUrl known at boot: bake the resource URLs once, no Host-header trust.\n if (config.baseUrl !== undefined) {\n let baseUrl: URL;\n try {\n baseUrl = new URL(config.baseUrl);\n } catch {\n throw new Error(\n `oauth.baseUrl must be a valid absolute URL, got: ${JSON.stringify(\n config.baseUrl,\n )}`,\n );\n }\n app.use(\n mcpAuthMetadataRouter({\n oauthMetadata: config.oauthMetadata,\n resourceServerUrl: baseUrl,\n scopesSupported: config.scopesSupported,\n }),\n );\n const url = getOAuthProtectedResourceMetadataUrl(baseUrl);\n resourceMetadataUrl = () => url;\n } else {\n // No baseUrl: resolve the resource origin per request from forwarded headers\n // (same precedence the framework uses for view serverUrl). Origin headers are\n // pathless, so the PRM path stays at root, matching the SDK's static layout.\n const resolveOrigin = (getHeader: (key: string) => string | undefined) =>\n new URL(resolveServerOrigin(getHeader));\n app.use(\n \"/.well-known/oauth-authorization-server\",\n cors(),\n (_req, res) => void res.json(config.oauthMetadata),\n );\n app.use(\"/.well-known/oauth-protected-resource\", cors(), (req, res) => {\n res.json({\n resource: resolveOrigin((key) => req.get(key)).href,\n authorization_servers: [config.oauthMetadata.issuer],\n scopes_supported: config.scopesSupported,\n });\n });\n resourceMetadataUrl = (getHeader) =>\n getOAuthProtectedResourceMetadataUrl(resolveOrigin(getHeader));\n }\n\n app.use(\"/mcp\", (req, res, next) =>\n bearer({\n verifier,\n requiredScopes: config.requiredScopes,\n resourceMetadataUrl: resourceMetadataUrl((key) => req.get(key)),\n })(req, res, next),\n );\n\n app.use(\"/mcp\", (req, res, next) => {\n type Call = {\n id?: string | number | null;\n method?: string;\n params?: { name?: string };\n };\n const body = req.body as Call | Call[] | undefined;\n const calls = Array.isArray(body) ? body : [body];\n const auth = (req as Request & { auth?: AuthInfo }).auth;\n let failure: SchemeFailure | undefined;\n for (const call of calls) {\n const tool = call?.params?.name;\n if (call?.method === \"tools/call\" && tool && schemesByTool.has(tool)) {\n failure = evaluateSecuritySchemes(schemesByTool.get(tool), auth);\n if (failure) {\n break;\n }\n }\n }\n if (!failure) {\n return next();\n }\n const url = resourceMetadataUrl((key) => req.get(key));\n if (\n !Array.isArray(body) &&\n clientPrefersInBandChallenge(req.get(\"user-agent\"))\n ) {\n res.json({\n jsonrpc: \"2.0\",\n id: body?.id ?? null,\n result: inBandChallengeResult(failure, url),\n });\n return;\n }\n res.set(\"WWW-Authenticate\", wwwAuthenticateHeader(failure, url));\n res.status(httpStatusForFailure(failure)).json({\n error: failure.error,\n error_description: failure.description,\n });\n });\n\n return resourceMetadataUrl;\n}\n"]}
|
|
@@ -5,6 +5,7 @@ import { StreamableHTTPClientTransport } from "@modelcontextprotocol/sdk/client/
|
|
|
5
5
|
import * as jose from "jose";
|
|
6
6
|
import { afterEach, describe, expect, it, vi } from "vitest";
|
|
7
7
|
import { McpServer } from "../server.js";
|
|
8
|
+
import { createJwksVerifier } from "./verify.js";
|
|
8
9
|
vi.mock("@skybridge/devtools", () => ({
|
|
9
10
|
devtoolsStaticServer: () => ((_req, _res, next) => next()),
|
|
10
11
|
}));
|
|
@@ -36,10 +37,10 @@ async function startJwks() {
|
|
|
36
37
|
const port = server.address().port;
|
|
37
38
|
return { privateKey, jwksUri: `http://localhost:${port}/jwks` };
|
|
38
39
|
}
|
|
39
|
-
function signToken(key) {
|
|
40
|
+
function signToken(key, scope = "openid email") {
|
|
40
41
|
return new jose.SignJWT({
|
|
41
42
|
client_id: "client-1",
|
|
42
|
-
scope
|
|
43
|
+
scope,
|
|
43
44
|
sub: "user-1",
|
|
44
45
|
})
|
|
45
46
|
.setProtectedHeader({ alg: "RS256", kid: "test-key" })
|
|
@@ -48,7 +49,7 @@ function signToken(key) {
|
|
|
48
49
|
.setExpirationTime("1h")
|
|
49
50
|
.sign(key);
|
|
50
51
|
}
|
|
51
|
-
async function bootServer(jwksUri, { baseUrl = "https://app.example.test" } = {}) {
|
|
52
|
+
async function bootServer(jwksUri, { baseUrl = "https://app.example.test", legacyVerify = false, } = {}) {
|
|
52
53
|
const { createApp } = await import("../express.js");
|
|
53
54
|
const server = new McpServer({ name: "auth-test", version: "0.0.0" }, { capabilities: {} }, {
|
|
54
55
|
oauth: {
|
|
@@ -59,7 +60,15 @@ async function bootServer(jwksUri, { baseUrl = "https://app.example.test" } = {}
|
|
|
59
60
|
token_endpoint: `${ISSUER}/token`,
|
|
60
61
|
response_types_supported: ["code"],
|
|
61
62
|
},
|
|
62
|
-
|
|
63
|
+
...(legacyVerify
|
|
64
|
+
? { verify: { issuer: ISSUER, audience: AUDIENCE, jwksUri } }
|
|
65
|
+
: {
|
|
66
|
+
verifier: createJwksVerifier({
|
|
67
|
+
issuer: ISSUER,
|
|
68
|
+
audience: AUDIENCE,
|
|
69
|
+
jwksUri,
|
|
70
|
+
}),
|
|
71
|
+
}),
|
|
63
72
|
scopesSupported: ["openid", "email"],
|
|
64
73
|
requiredScopes: ["openid"],
|
|
65
74
|
},
|
|
@@ -115,6 +124,20 @@ describe("setupOAuth wiring", () => {
|
|
|
115
124
|
expect(result.content[0]?.text).toBe("client-1");
|
|
116
125
|
await client.close();
|
|
117
126
|
});
|
|
127
|
+
it("builds the verifier from the legacy verify config", async () => {
|
|
128
|
+
const { privateKey, jwksUri } = await startJwks();
|
|
129
|
+
const base = await bootServer(jwksUri, { legacyVerify: true });
|
|
130
|
+
const token = await signToken(privateKey);
|
|
131
|
+
const client = new Client({ name: "test-client", version: "0.0.0" });
|
|
132
|
+
const transport = new StreamableHTTPClientTransport(new URL(`${base}/mcp`), { requestInit: { headers: { Authorization: `Bearer ${token}` } } });
|
|
133
|
+
await client.connect(transport);
|
|
134
|
+
const result = (await client.callTool({
|
|
135
|
+
name: "whoami",
|
|
136
|
+
arguments: {},
|
|
137
|
+
}));
|
|
138
|
+
expect(result.content[0]?.text).toBe("client-1");
|
|
139
|
+
await client.close();
|
|
140
|
+
});
|
|
118
141
|
});
|
|
119
142
|
describe("baseUrl inferred from headers", () => {
|
|
120
143
|
it("derives the resource origin from x-forwarded-host", async () => {
|
|
@@ -165,6 +188,303 @@ describe("baseUrl inferred from headers", () => {
|
|
|
165
188
|
expect(res.headers.get("www-authenticate")).toMatch(/resource_metadata="https:\/\/infer\.example\.test\//);
|
|
166
189
|
});
|
|
167
190
|
});
|
|
191
|
+
async function bootMixedServer(jwksUri) {
|
|
192
|
+
const { createApp } = await import("../express.js");
|
|
193
|
+
const server = new McpServer({ name: "mixed-auth-test", version: "0.0.0" }, { capabilities: {} }, {
|
|
194
|
+
oauth: {
|
|
195
|
+
baseUrl: "https://app.example.test",
|
|
196
|
+
oauthMetadata: {
|
|
197
|
+
issuer: ISSUER,
|
|
198
|
+
authorization_endpoint: `${ISSUER}/authorize`,
|
|
199
|
+
token_endpoint: `${ISSUER}/token`,
|
|
200
|
+
response_types_supported: ["code"],
|
|
201
|
+
},
|
|
202
|
+
verifier: createJwksVerifier({
|
|
203
|
+
issuer: ISSUER,
|
|
204
|
+
audience: AUDIENCE,
|
|
205
|
+
jwksUri,
|
|
206
|
+
}),
|
|
207
|
+
},
|
|
208
|
+
})
|
|
209
|
+
.registerTool({
|
|
210
|
+
name: "public-whoami",
|
|
211
|
+
description: "Public.",
|
|
212
|
+
inputSchema: {},
|
|
213
|
+
auth: { allowsAnonymous: true },
|
|
214
|
+
}, (_args, extra) => ({
|
|
215
|
+
content: [{ type: "text", text: extra.authInfo?.clientId ?? "anon" }],
|
|
216
|
+
}))
|
|
217
|
+
.registerTool({
|
|
218
|
+
name: "private-whoami",
|
|
219
|
+
description: "Private.",
|
|
220
|
+
inputSchema: {},
|
|
221
|
+
auth: {},
|
|
222
|
+
}, (_args, extra) => ({
|
|
223
|
+
content: [{ type: "text", text: extra.authInfo?.clientId ?? "anon" }],
|
|
224
|
+
}));
|
|
225
|
+
server.registerTool("legacy-whoami", {
|
|
226
|
+
description: "Registered via the legacy string overload.",
|
|
227
|
+
inputSchema: {},
|
|
228
|
+
}, (_args, extra) => ({
|
|
229
|
+
content: [{ type: "text", text: extra.authInfo?.clientId ?? "anon" }],
|
|
230
|
+
}));
|
|
231
|
+
const httpServer = http.createServer();
|
|
232
|
+
await createApp({ mcpServer: server, httpServer });
|
|
233
|
+
const listening = http.createServer(server.express);
|
|
234
|
+
await new Promise((resolve) => listening.listen(0, resolve));
|
|
235
|
+
appServer = listening;
|
|
236
|
+
const port = listening.address().port;
|
|
237
|
+
return `http://localhost:${port}`;
|
|
238
|
+
}
|
|
239
|
+
describe("mixed-auth door", () => {
|
|
240
|
+
it("lets an anonymous caller initialize and run a noauth tool", async () => {
|
|
241
|
+
const { jwksUri } = await startJwks();
|
|
242
|
+
const base = await bootMixedServer(jwksUri);
|
|
243
|
+
const client = new Client({ name: "test-client", version: "0.0.0" });
|
|
244
|
+
await client.connect(new StreamableHTTPClientTransport(new URL(`${base}/mcp`)));
|
|
245
|
+
const result = (await client.callTool({
|
|
246
|
+
name: "public-whoami",
|
|
247
|
+
arguments: {},
|
|
248
|
+
}));
|
|
249
|
+
expect(result.content[0]?.text).toBe("anon");
|
|
250
|
+
await client.close();
|
|
251
|
+
});
|
|
252
|
+
it("exposes securitySchemes at the top level of tools/list (not only _meta)", async () => {
|
|
253
|
+
const { jwksUri } = await startJwks();
|
|
254
|
+
const base = await bootMixedServer(jwksUri);
|
|
255
|
+
const headers = {
|
|
256
|
+
"Content-Type": "application/json",
|
|
257
|
+
Accept: "application/json, text/event-stream",
|
|
258
|
+
};
|
|
259
|
+
await fetch(`${base}/mcp`, {
|
|
260
|
+
method: "POST",
|
|
261
|
+
headers,
|
|
262
|
+
body: JSON.stringify({
|
|
263
|
+
jsonrpc: "2.0",
|
|
264
|
+
id: 1,
|
|
265
|
+
method: "initialize",
|
|
266
|
+
params: {
|
|
267
|
+
protocolVersion: "2025-06-18",
|
|
268
|
+
capabilities: {},
|
|
269
|
+
clientInfo: { name: "c", version: "0" },
|
|
270
|
+
},
|
|
271
|
+
}),
|
|
272
|
+
});
|
|
273
|
+
const res = await fetch(`${base}/mcp`, {
|
|
274
|
+
method: "POST",
|
|
275
|
+
headers,
|
|
276
|
+
body: JSON.stringify({ jsonrpc: "2.0", id: 2, method: "tools/list" }),
|
|
277
|
+
});
|
|
278
|
+
const text = await res.text();
|
|
279
|
+
const json = JSON.parse((text.match(/\{[\s\S]*\}/) ?? ["{}"])[0]);
|
|
280
|
+
const whoami = json.result.tools.find((t) => t.name === "private-whoami");
|
|
281
|
+
expect(whoami?.securitySchemes).toEqual([{ type: "oauth2" }]);
|
|
282
|
+
});
|
|
283
|
+
it("denies a protected tool inside an anonymous JSON-RPC batch", async () => {
|
|
284
|
+
const { jwksUri } = await startJwks();
|
|
285
|
+
const base = await bootMixedServer(jwksUri);
|
|
286
|
+
const res = await fetch(`${base}/mcp`, {
|
|
287
|
+
method: "POST",
|
|
288
|
+
headers: {
|
|
289
|
+
"Content-Type": "application/json",
|
|
290
|
+
Accept: "application/json, text/event-stream",
|
|
291
|
+
},
|
|
292
|
+
body: JSON.stringify([
|
|
293
|
+
{
|
|
294
|
+
jsonrpc: "2.0",
|
|
295
|
+
id: 1,
|
|
296
|
+
method: "tools/call",
|
|
297
|
+
params: { name: "private-whoami", arguments: {} },
|
|
298
|
+
},
|
|
299
|
+
]),
|
|
300
|
+
});
|
|
301
|
+
expect(res.status).toBe(401);
|
|
302
|
+
const wwwAuthenticate = res.headers.get("www-authenticate") ?? "";
|
|
303
|
+
expect(wwwAuthenticate).toMatch(/^Bearer /);
|
|
304
|
+
expect(wwwAuthenticate).toContain("resource_metadata=");
|
|
305
|
+
const text = await res.text();
|
|
306
|
+
expect(text).not.toContain("client-1");
|
|
307
|
+
});
|
|
308
|
+
it("gates a tool registered via the legacy string overload (secure default)", async () => {
|
|
309
|
+
const { jwksUri } = await startJwks();
|
|
310
|
+
const base = await bootMixedServer(jwksUri);
|
|
311
|
+
const res = await fetch(`${base}/mcp`, {
|
|
312
|
+
method: "POST",
|
|
313
|
+
headers: {
|
|
314
|
+
"Content-Type": "application/json",
|
|
315
|
+
Accept: "application/json, text/event-stream",
|
|
316
|
+
},
|
|
317
|
+
body: JSON.stringify({
|
|
318
|
+
jsonrpc: "2.0",
|
|
319
|
+
id: 1,
|
|
320
|
+
method: "tools/call",
|
|
321
|
+
params: { name: "legacy-whoami", arguments: {} },
|
|
322
|
+
}),
|
|
323
|
+
});
|
|
324
|
+
expect(res.status).toBe(401);
|
|
325
|
+
});
|
|
326
|
+
it("returns 401 + WWW-Authenticate for a protected tool while anonymous", async () => {
|
|
327
|
+
const { jwksUri } = await startJwks();
|
|
328
|
+
const base = await bootMixedServer(jwksUri);
|
|
329
|
+
const res = await fetch(`${base}/mcp`, {
|
|
330
|
+
method: "POST",
|
|
331
|
+
headers: {
|
|
332
|
+
"Content-Type": "application/json",
|
|
333
|
+
Accept: "application/json, text/event-stream",
|
|
334
|
+
},
|
|
335
|
+
body: JSON.stringify({
|
|
336
|
+
jsonrpc: "2.0",
|
|
337
|
+
id: 1,
|
|
338
|
+
method: "tools/call",
|
|
339
|
+
params: { name: "private-whoami", arguments: {} },
|
|
340
|
+
}),
|
|
341
|
+
});
|
|
342
|
+
expect(res.status).toBe(401);
|
|
343
|
+
expect(res.headers.get("www-authenticate")).toMatch(/error="invalid_token"/);
|
|
344
|
+
expect(res.headers.get("www-authenticate")).toMatch(/resource_metadata=/);
|
|
345
|
+
});
|
|
346
|
+
it("returns an in-band mcp/www_authenticate array to ChatGPT (not a transport 401)", async () => {
|
|
347
|
+
const { jwksUri } = await startJwks();
|
|
348
|
+
const base = await bootMixedServer(jwksUri);
|
|
349
|
+
const res = await fetch(`${base}/mcp`, {
|
|
350
|
+
method: "POST",
|
|
351
|
+
headers: {
|
|
352
|
+
"Content-Type": "application/json",
|
|
353
|
+
Accept: "application/json, text/event-stream",
|
|
354
|
+
"User-Agent": "openai-mcp/1.0",
|
|
355
|
+
},
|
|
356
|
+
body: JSON.stringify({
|
|
357
|
+
jsonrpc: "2.0",
|
|
358
|
+
id: 7,
|
|
359
|
+
method: "tools/call",
|
|
360
|
+
params: { name: "private-whoami", arguments: {} },
|
|
361
|
+
}),
|
|
362
|
+
});
|
|
363
|
+
expect(res.status).toBe(200);
|
|
364
|
+
expect(res.headers.get("www-authenticate")).toBeNull();
|
|
365
|
+
const body = (await res.json());
|
|
366
|
+
expect(body.id).toBe(7);
|
|
367
|
+
expect(body.result.isError).toBe(true);
|
|
368
|
+
const challenge = body.result._meta["mcp/www_authenticate"];
|
|
369
|
+
expect(Array.isArray(challenge)).toBe(true);
|
|
370
|
+
expect(challenge[0]).toMatch(/error="invalid_token"/);
|
|
371
|
+
expect(challenge[0]).toMatch(/resource_metadata=/);
|
|
372
|
+
});
|
|
373
|
+
it("fails closed on a ChatGPT batch (transport 401, no in-band challenge)", async () => {
|
|
374
|
+
const { jwksUri } = await startJwks();
|
|
375
|
+
const base = await bootMixedServer(jwksUri);
|
|
376
|
+
const res = await fetch(`${base}/mcp`, {
|
|
377
|
+
method: "POST",
|
|
378
|
+
headers: {
|
|
379
|
+
"Content-Type": "application/json",
|
|
380
|
+
Accept: "application/json, text/event-stream",
|
|
381
|
+
"User-Agent": "openai-mcp/1.0",
|
|
382
|
+
},
|
|
383
|
+
body: JSON.stringify([
|
|
384
|
+
{
|
|
385
|
+
jsonrpc: "2.0",
|
|
386
|
+
id: 1,
|
|
387
|
+
method: "tools/call",
|
|
388
|
+
params: { name: "private-whoami", arguments: {} },
|
|
389
|
+
},
|
|
390
|
+
]),
|
|
391
|
+
});
|
|
392
|
+
expect(res.status).toBe(401);
|
|
393
|
+
expect(res.headers.get("www-authenticate")).toMatch(/^Bearer /);
|
|
394
|
+
const text = await res.text();
|
|
395
|
+
expect(text).not.toContain("client-1");
|
|
396
|
+
});
|
|
397
|
+
});
|
|
398
|
+
async function bootScopedServer(jwksUri) {
|
|
399
|
+
const { createApp } = await import("../express.js");
|
|
400
|
+
const server = new McpServer({ name: "scoped-auth-test", version: "0.0.0" }, { capabilities: {} }, {
|
|
401
|
+
oauth: {
|
|
402
|
+
baseUrl: "https://app.example.test",
|
|
403
|
+
oauthMetadata: {
|
|
404
|
+
issuer: ISSUER,
|
|
405
|
+
authorization_endpoint: `${ISSUER}/authorize`,
|
|
406
|
+
token_endpoint: `${ISSUER}/token`,
|
|
407
|
+
response_types_supported: ["code"],
|
|
408
|
+
},
|
|
409
|
+
verifier: createJwksVerifier({
|
|
410
|
+
issuer: ISSUER,
|
|
411
|
+
audience: AUDIENCE,
|
|
412
|
+
jwksUri,
|
|
413
|
+
}),
|
|
414
|
+
},
|
|
415
|
+
}).registerTool({
|
|
416
|
+
name: "checkout",
|
|
417
|
+
description: "Needs the checkout scope.",
|
|
418
|
+
inputSchema: {},
|
|
419
|
+
auth: { scopes: ["checkout"] },
|
|
420
|
+
}, () => ({ content: [{ type: "text", text: "ok" }] }));
|
|
421
|
+
const httpServer = http.createServer();
|
|
422
|
+
await createApp({ mcpServer: server, httpServer });
|
|
423
|
+
const listening = http.createServer(server.express);
|
|
424
|
+
await new Promise((resolve) => listening.listen(0, resolve));
|
|
425
|
+
appServer = listening;
|
|
426
|
+
const port = listening.address().port;
|
|
427
|
+
return `http://localhost:${port}`;
|
|
428
|
+
}
|
|
429
|
+
describe("per-tool scope enforcement in fully-authenticated mode", () => {
|
|
430
|
+
it("returns 403 for a valid token missing the tool's scope", async () => {
|
|
431
|
+
const { privateKey, jwksUri } = await startJwks();
|
|
432
|
+
const base = await bootScopedServer(jwksUri);
|
|
433
|
+
const token = await signToken(privateKey, "openid");
|
|
434
|
+
const res = await fetch(`${base}/mcp`, {
|
|
435
|
+
method: "POST",
|
|
436
|
+
headers: {
|
|
437
|
+
"Content-Type": "application/json",
|
|
438
|
+
Accept: "application/json, text/event-stream",
|
|
439
|
+
Authorization: `Bearer ${token}`,
|
|
440
|
+
},
|
|
441
|
+
body: JSON.stringify({
|
|
442
|
+
jsonrpc: "2.0",
|
|
443
|
+
id: 1,
|
|
444
|
+
method: "tools/call",
|
|
445
|
+
params: { name: "checkout", arguments: {} },
|
|
446
|
+
}),
|
|
447
|
+
});
|
|
448
|
+
expect(res.status).toBe(403);
|
|
449
|
+
const header = res.headers.get("www-authenticate");
|
|
450
|
+
expect(header).toMatch(/error="insufficient_scope"/);
|
|
451
|
+
expect(header).toMatch(/scope="checkout"/);
|
|
452
|
+
});
|
|
453
|
+
it("allows a token carrying the tool's scope", async () => {
|
|
454
|
+
const { privateKey, jwksUri } = await startJwks();
|
|
455
|
+
const base = await bootScopedServer(jwksUri);
|
|
456
|
+
const token = await signToken(privateKey, "openid checkout");
|
|
457
|
+
const client = new Client({ name: "test-client", version: "0.0.0" });
|
|
458
|
+
await client.connect(new StreamableHTTPClientTransport(new URL(`${base}/mcp`), {
|
|
459
|
+
requestInit: { headers: { Authorization: `Bearer ${token}` } },
|
|
460
|
+
}));
|
|
461
|
+
const result = (await client.callTool({
|
|
462
|
+
name: "checkout",
|
|
463
|
+
arguments: {},
|
|
464
|
+
}));
|
|
465
|
+
expect(result.content[0]?.text).toBe("ok");
|
|
466
|
+
await client.close();
|
|
467
|
+
});
|
|
468
|
+
});
|
|
469
|
+
describe("auth shorthand validation", () => {
|
|
470
|
+
it("throws when `auth` requires sign-in but no oauth provider is configured", () => {
|
|
471
|
+
expect(() => new McpServer({ name: "t", version: "0" }).registerTool({ name: "x", inputSchema: {}, auth: {} }, () => ({ content: [{ type: "text", text: "" }] }))).toThrow(/no `oauth` provider/);
|
|
472
|
+
});
|
|
473
|
+
it("throws when `auth` sets scopes but no oauth provider is configured", () => {
|
|
474
|
+
expect(() => new McpServer({ name: "t", version: "0" }).registerTool({ name: "x", inputSchema: {}, auth: { scopes: ["checkout"] } }, () => ({ content: [{ type: "text", text: "" }] }))).toThrow(/no `oauth` provider/);
|
|
475
|
+
});
|
|
476
|
+
it("allows `auth: { allowsAnonymous: true }` without an oauth provider", () => {
|
|
477
|
+
expect(() => new McpServer({ name: "t", version: "0" }).registerTool({ name: "x", inputSchema: {}, auth: { allowsAnonymous: true } }, () => ({ content: [{ type: "text", text: "" }] }))).not.toThrow();
|
|
478
|
+
});
|
|
479
|
+
it("prefers `securitySchemes` over `auth` when both are set (untyped path)", () => {
|
|
480
|
+
expect(() => new McpServer({ name: "t", version: "0" }).registerTool({
|
|
481
|
+
name: "x",
|
|
482
|
+
inputSchema: {},
|
|
483
|
+
auth: {},
|
|
484
|
+
securitySchemes: [{ type: "oauth2" }],
|
|
485
|
+
}, () => ({ content: [{ type: "text", text: "" }] }))).not.toThrow();
|
|
486
|
+
});
|
|
487
|
+
});
|
|
168
488
|
describe("oauth config validation", () => {
|
|
169
489
|
const validMetadata = {
|
|
170
490
|
issuer: ISSUER,
|
|
@@ -177,7 +497,10 @@ describe("oauth config validation", () => {
|
|
|
177
497
|
oauth: {
|
|
178
498
|
baseUrl: "not-a-url",
|
|
179
499
|
oauthMetadata: validMetadata,
|
|
180
|
-
|
|
500
|
+
verifier: createJwksVerifier({
|
|
501
|
+
issuer: ISSUER,
|
|
502
|
+
audience: AUDIENCE,
|
|
503
|
+
}),
|
|
181
504
|
},
|
|
182
505
|
})).toThrow(/baseUrl must be a valid absolute URL/);
|
|
183
506
|
});
|