Add nullable attribute to AuthorizationResult.Succeeded (#46092)

Author: frederikrosenberg

Diff for: src/Security/Authorization/Core/src/AuthorizationResult.cs

@@ -2,6 +2,7 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 
 using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
 using System.Security.Claims;
 
 namespace Microsoft.AspNetCore.Authorization;
@@ -19,6 +20,7 @@ private AuthorizationResult() { }
     /// <summary>
     /// True if authorization was successful.
     /// </summary>
+    [MemberNotNullWhen(false, nameof(Failure))]
     public bool Succeeded { get; private set; }
 
     /// <summary>

Diff for: src/Security/Authorization/Core/src/DefaultAuthorizationService.cs

@@ -81,7 +81,7 @@ public virtual async Task<AuthorizationResult> AuthorizeAsync(ClaimsPrincipal us
         }
         else
         {
-            _logger.UserAuthorizationFailed(result.Failure!);
+            _logger.UserAuthorizationFailed(result.Failure);
         }
         return result;
     }