modelcontextprotocol/typescript-sdk
Watch
1
0
Fork
You've already forked typescript-sdk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Code review comments

Browse source
This commit is contained in:
Justin Spahr-Summers 2025-02-17 10:34:57 +00:00
commit 44a440854f
No known key found for this signature in database
2 changed files with 3 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/client/auth.test.ts
View file

@ -84,8 +84,8 @@ describe("OAuth Authorization", () => {
describe("startAuthorization", () => {
const validMetadata = {
issuer: "https://auth.example.com",
authorization_endpoint: "https://auth.example.com/authorize",
token_endpoint: "https://auth.example.com/token",
authorization_endpoint: "https://auth.example.com/auth",
token_endpoint: "https://auth.example.com/tkn",
response_types_supported: ["code"],
code_challenge_methods_supported: ["S256"],
};
@ -131,7 +131,7 @@ describe("OAuth Authorization", () => {
);
expect(authorizationUrl.toString()).toMatch(
/^https:\/\/auth\.example\.com\/authorize\?/
/^https:\/\/auth\.example\.com\/auth\?/
);
});

7
src/client/auth.ts
View file

@ -432,13 +432,6 @@ export async function refreshAuthorization(
/**
* Performs OAuth 2.0 Dynamic Client Registration according to RFC 7591.
*
* @param serverUrl - The base URL of the authorization server
* @param options - Registration options
* @param options.metadata - OAuth server metadata containing the registration endpoint
* @param options.clientMetadata - Client metadata for registration
* @returns The registered client information
* @throws Error if the server doesn't support dynamic registration or if registration fails
*/
export async function registerClient(
serverUrl: string | URL,