Skip to content

Commit 3562966

Browse files
authored
fix(auth): sanitize authorization URL (#16)
1 parent 10e3cc3 commit 3562966

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@
3030
"prompts": "^2.4.2",
3131
"uri-template": "^2.0.0",
3232
"yocto-spinner": "^0.2.2",
33-
"yoctocolors": "^2.1.1"
33+
"yoctocolors": "^2.1.1",
34+
"strict-url-sanitise": "^0.0.1"
3435
}
3536
}

src/oauth/provider.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
import open from 'open'
44
import { config } from '../config.js'
5+
import { sanitizeUrl } from 'strict-url-sanitise'
56

67
/** @typedef {import("@modelcontextprotocol/sdk/client/auth.js").OAuthClientProvider} OAuthClientProvider */
78
/** @implements {OAuthClientProvider} */
@@ -39,7 +40,7 @@ export class McpOAuthClientProvider {
3940
}
4041

4142
async redirectToAuthorization(authorizationUrl) {
42-
await open(authorizationUrl.toString())
43+
await open(sanitizeUrl(authorizationUrl.toString()))
4344
}
4445

4546
async codeVerifier() {

0 commit comments

Comments
 (0)