Skip to content

Commit 18ea479

Browse files
authored
Apply suggestions from code review
1 parent e5260c8 commit 18ea479

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/Security/Authorization/Core/src/AuthorizationServiceExtensions.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ public static class AuthorizationServiceExtensions
2222
/// <param name="requirement">The requirement to evaluate the policy against.</param>
2323
/// <returns>
2424
/// A <see cref="Task{TResult}"/> that contains an <see cref="AuthorizationResult"/> indicating whether requirement evaluation has succeeded or failed.
25-
/// The result's <see cref="AuthorizationResult.Succeeded"/> property is <c>true</c> when the user fulfills the policy, otherwise <c>false</c>.
25+
/// The result's <see cref="AuthorizationResult.Succeeded"/> property is <c>true</c> when the user fulfills the policy; otherwise <c>false</c>.
2626
/// </returns>
2727
public static Task<AuthorizationResult> AuthorizeAsync(this IAuthorizationService service, ClaimsPrincipal user, object? resource, IAuthorizationRequirement requirement)
2828
{
@@ -41,7 +41,7 @@ public static Task<AuthorizationResult> AuthorizeAsync(this IAuthorizationServic
4141
/// <param name="policy">The policy to evaluate.</param>
4242
/// <returns>
4343
/// A <see cref="Task{TResult}"/> that contains an <see cref="AuthorizationResult"/> indicating whether policy evaluation has succeeded or failed.
44-
/// The result's <see cref="AuthorizationResult.Succeeded"/> property is <c>true</c> when the user fulfills the policy, otherwise <c>false</c>.
44+
/// The result's <see cref="AuthorizationResult.Succeeded"/> property is <c>true</c> when the user fulfills the policy; otherwise <c>false</c>.
4545
/// </returns>
4646
public static Task<AuthorizationResult> AuthorizeAsync(this IAuthorizationService service, ClaimsPrincipal user, object? resource, AuthorizationPolicy policy)
4747
{
@@ -59,7 +59,7 @@ public static Task<AuthorizationResult> AuthorizeAsync(this IAuthorizationServic
5959
/// <param name="policy">The policy to evaluate.</param>
6060
/// <returns>
6161
/// A <see cref="Task{TResult}"/> that contains an <see cref="AuthorizationResult"/> indicating whether policy evaluation has succeeded or failed.
62-
/// The result's <see cref="AuthorizationResult.Succeeded"/> property is <c>true</c> when the user fulfills the policy, otherwise <c>false</c>.
62+
/// The result's <see cref="AuthorizationResult.Succeeded"/> property is <c>true</c> when the user fulfills the policy; otherwise <c>false</c>.
6363
/// </returns>
6464
public static Task<AuthorizationResult> AuthorizeAsync(this IAuthorizationService service, ClaimsPrincipal user, AuthorizationPolicy policy)
6565
{
@@ -77,7 +77,7 @@ public static Task<AuthorizationResult> AuthorizeAsync(this IAuthorizationServic
7777
/// <param name="policyName">The name of the policy to evaluate.</param>
7878
/// <returns>
7979
/// A <see cref="Task{TResult}"/> that contains an <see cref="AuthorizationResult"/> indicating whether policy evaluation has succeeded or failed.
80-
/// The result's <see cref="AuthorizationResult.Succeeded"/> property is <c>true</c> when the user fulfills the policy, otherwise <c>false</c>.
80+
/// The result's <see cref="AuthorizationResult.Succeeded"/> property is <c>true</c> when the user fulfills the policy; otherwise <c>false</c>.
8181
/// </returns>
8282
public static Task<AuthorizationResult> AuthorizeAsync(this IAuthorizationService service, ClaimsPrincipal user, string policyName)
8383
{

0 commit comments

Comments
 (0)