Skip to content

Commit bf6c1c4

Browse files
authored
Warn that TokenValidated is not the last step of validation (#32299)
1 parent 5c3ef58 commit bf6c1c4

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/Security/Authentication/OpenIdConnect/src/Events/OpenIdConnectEvents.cs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,8 @@ public class OpenIdConnectEvents : RemoteAuthenticationEvents
5454
public Func<TokenResponseReceivedContext, Task> OnTokenResponseReceived { get; set; } = context => Task.CompletedTask;
5555

5656
/// <summary>
57-
/// Invoked when an IdToken has been validated and produced an AuthenticationTicket.
57+
/// Invoked when an IdToken has been validated and produced an AuthenticationTicket. Note there are additional checks after this
58+
/// event that validate other aspects of the authentication flow like the nonce.
5859
/// </summary>
5960
public Func<TokenValidatedContext, Task> OnTokenValidated { get; set; } = context => Task.CompletedTask;
6061

@@ -106,7 +107,8 @@ public class OpenIdConnectEvents : RemoteAuthenticationEvents
106107
public virtual Task TokenResponseReceived(TokenResponseReceivedContext context) => OnTokenResponseReceived(context);
107108

108109
/// <summary>
109-
/// Invoked when an IdToken has been validated and produced an AuthenticationTicket.
110+
/// Invoked when an IdToken has been validated and produced an AuthenticationTicket. Note there are additional checks after this
111+
/// event that validate other aspects of the authentication flow like the nonce.
110112
/// </summary>
111113
public virtual Task TokenValidated(TokenValidatedContext context) => OnTokenValidated(context);
112114

0 commit comments

Comments
 (0)