Skip to content

Commit 864e3ae

Browse files
committed
fix: use didUrl instead of kid for jwt signer did
Signed-off-by: Timo Glastra <[email protected]>
1 parent d9b8118 commit 864e3ae

File tree

1 file changed

+3
-9
lines changed

1 file changed

+3
-9
lines changed

packages/openid4vp/src/client-identifier-scheme/parse-client-identifier-scheme.ts

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -304,14 +304,8 @@ export function validateOpenid4vpClientId(
304304
})
305305
}
306306

307-
if (!jar.signer.publicJwk.kid) {
308-
throw new Oauth2ServerErrorResponseError({
309-
error: Oauth2ErrorCodes.InvalidRequest,
310-
error_description: `Missing required 'kid' for client identifier scheme: did`,
311-
})
312-
}
313-
314-
if (!jar.signer.publicJwk.kid?.startsWith(clientId)) {
307+
const [did] = jar.signer.didUrl.split('#')
308+
if (clientId !== did) {
315309
throw new Oauth2ServerErrorResponseError({
316310
error: Oauth2ErrorCodes.InvalidRequest,
317311
error_description:
@@ -324,7 +318,7 @@ export function validateOpenid4vpClientId(
324318
identifier: clientId,
325319
originalValue: clientId,
326320
legacyClientId,
327-
didUrl: jar.signer.publicJwk.kid,
321+
didUrl: jar.signer.didUrl,
328322
}
329323
}
330324

0 commit comments

Comments
 (0)