Check AuthenticateResult of IAuthenticateResultFeature is not null. (#1918)

maliming · web-flow · commit fa153181fa8e · 2024-03-01T03:58:14.000Z
diff --git a/src/Microsoft.Azure.SignalR/HubHost/NegotiateHandler.cs b/src/Microsoft.Azure.SignalR/HubHost/NegotiateHandler.cs
@@ -138,7 +138,7 @@ private IEnumerable<Claim> BuildClaims(HttpContext context)
 #if NET6_0_OR_GREATER
             closeOnAuthenticationExpiration = _dispatcherOptions.CloseOnAuthenticationExpiration;
             var authResultFeature = context.Features.Get<IAuthenticateResultFeature>();
-            if (authResultFeature != null && authResultFeature.AuthenticateResult.Succeeded)
+            if (authResultFeature != null && authResultFeature.AuthenticateResult != null && authResultFeature.AuthenticateResult.Succeeded)
             {
                 authenticationExpiresOn = authResultFeature.AuthenticateResult.Properties.ExpiresUtc;
             }

Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ private IEnumerable<Claim> BuildClaims(HttpContext context)`
`138`	`138`	`#if NET6_0_OR_GREATER`
`139`	`139`	`closeOnAuthenticationExpiration = _dispatcherOptions.CloseOnAuthenticationExpiration;`
`140`	`140`	`var authResultFeature = context.Features.Get<IAuthenticateResultFeature>();`
`141`		`- if (authResultFeature != null && authResultFeature.AuthenticateResult.Succeeded)`
	`141`	`+ if (authResultFeature != null && authResultFeature.AuthenticateResult != null && authResultFeature.AuthenticateResult.Succeeded)`
`142`	`142`	`{`
`143`	`143`	`authenticationExpiresOn = authResultFeature.AuthenticateResult.Properties.ExpiresUtc;`
`144`	`144`	`}`