Skip to content

Commit 03deb88

Browse files
committed
Fix build and lint issues
- Corrected imports in external-verifier.ts (OAuthTokenVerifier from provider.js) - Fixed InvalidTokenError import from errors.js - Changed logger.warn to logger.info (warn method doesn't exist) - Removed unused OAuthTokens import from services/auth.ts
1 parent 0135abd commit 03deb88

File tree

2 files changed

+6
-7
lines changed

2 files changed

+6
-7
lines changed

src/auth/external-verifier.ts

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
1-
import { OAuthTokenVerifier } from '@modelcontextprotocol/sdk/server/auth/types.js';
2-
import { AuthInfo, InvalidTokenError } from '@modelcontextprotocol/sdk/server/auth/types.js';
1+
import { OAuthTokenVerifier } from '@modelcontextprotocol/sdk/server/auth/provider.js';
2+
import { AuthInfo } from '@modelcontextprotocol/sdk/server/auth/types.js';
3+
import { InvalidTokenError } from '@modelcontextprotocol/sdk/server/auth/errors.js';
34
import { TokenIntrospectionResponse } from '../../shared/types.js';
45
import { logger } from '../utils/logger.js';
56

@@ -53,10 +54,8 @@ export class ExternalAuthVerifier implements OAuthTokenVerifier {
5354
// Extract user ID from standard 'sub' claim or custom 'userId' field
5455
const userId = data.sub || data.userId;
5556
if (!userId) {
56-
logger.warn('Token introspection response missing user ID', {
57-
hasS
58-
59-
: !!data.sub,
57+
logger.info('Token introspection response missing user ID', {
58+
hasSub: !!data.sub,
6059
hasUserId: !!data.userId,
6160
});
6261
}

src/services/auth.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { redisClient } from "../redis.js";
22
import { McpInstallation, PendingAuthorization, TokenExchange } from "../types.js";
3-
import { OAuthClientInformationFull, OAuthTokens } from "@modelcontextprotocol/sdk/shared/auth.js";
3+
import { OAuthClientInformationFull } from "@modelcontextprotocol/sdk/shared/auth.js";
44

55
// Re-export from shared modules for backward compatibility
66
export {

0 commit comments

Comments
 (0)