huahaofeng
/
aspnetcore
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Change description of role in RequireRole method 

The description of the pram roles feels ambiguous. Make it clear that the user should belong to one of the roles and not all of them.
As documented: https://docs.microsoft.com/en-us/aspnet/core/security/authorization/roles?view=aspnetcore-2.1
This commit is contained in:
Wouter 2018-07-31 13:40:49 +02:00 committed by GitHub
parent 2ddfd9ec46
commit 1e14a22e0d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/Microsoft.AspNetCore.Authorization/AuthorizationPolicyBuilder.cs
View File

@ -147,7 +147,7 @@ namespace Microsoft.AspNetCore.Authorization
/// Adds a <see cref="RolesAuthorizationRequirement"/>
/// to the current instance.
/// </summary>
/// <param name="roles">The roles required.</param>
/// <param name="roles">The allowed roles.</param>
/// <returns>A reference to this instance after the operation has completed.</returns>
public AuthorizationPolicyBuilder RequireRole(params string[] roles)
{
@ -163,7 +163,7 @@ namespace Microsoft.AspNetCore.Authorization
/// Adds a <see cref="RolesAuthorizationRequirement"/>
/// to the current instance.
/// </summary>
/// <param name="roles">The roles required.</param>
/// <param name="roles">The allowed roles.</param>
/// <returns>A reference to this instance after the operation has completed.</returns>
public AuthorizationPolicyBuilder RequireRole(IEnumerable<string> roles)
{
@ -247,4 +247,4 @@ namespace Microsoft.AspNetCore.Authorization
return new AuthorizationPolicy(Requirements, AuthenticationSchemes.Distinct());
}
}
}
}