-
Notifications
You must be signed in to change notification settings - Fork 10.4k
Closed as not planned
Closed as not planned
Copy link
Labels
area-authIncludes: Authn, Authz, OAuth, OIDC, BearerIncludes: Authn, Authz, OAuth, OIDC, Bearerinvestigate
Milestone
Description
Hello,
I've noticed that the OpenIdConnectHandler
is setting TokenValidationParameters.RequireSignedTokens
to false
in case of authorization code flow as per OIDC spec:
aspnetcore/src/Security/Authentication/OpenIdConnect/src/OpenIdConnectHandler.cs
Lines 685 to 687 in 2924ca2
// no need to validate signature when token is received using "code flow" as per spec | |
// [http://openid.net/specs/openid-connect-core-1_0.html#IDTokenValidation]. | |
validationParameters.RequireSignedTokens = false; |
However the JwtSecurityTokenHandler.ValidateSignature()
method that, if I'm not wrong, eventually gets called, validates the signature when the token is signed even if TokenValidationParameters.RequireSignedTokens
is false
.
Am I missing something?
Is this something expected in the OpenIdConnectHandler
?
Thank you very much.
Metadata
Metadata
Assignees
Labels
area-authIncludes: Authn, Authz, OAuth, OIDC, BearerIncludes: Authn, Authz, OAuth, OIDC, Bearerinvestigate