授权具有多个角色的属性

Question

我想为一个控制器添加授权，一次用于多个角色。

通常看起来像这样：

[Authorize(Roles = "RoleA,RoleB,RoleC")]
public async Task<ActionResult> Index()
{
}

但我已将我的角色存储在 const 中，因为它们可能会在某些时候更改或扩展。

public const RoleA = "RoleA";
public const RoleB = "RoleB";
public const RoleC = "RoleC";

我不能这样做，因为字符串必须在编译时知道：

[Authorize(Roles = string.join(",",RoleA,RoleB,RoleC)]
public async Task<ActionResult> Index()
{
}

有没有办法绕过这个问题？

我可以编写一个仅包含“RoleA,RoleB,RoleC”的 const - 但我不喜欢魔术字符串，这是一个魔术字符串。更改角色的名称而忘记更改组合字符串将是一场灾难。

我正在使用 MVC5。 ASP.NET 标识和角色在编译时是已知的。

Answer 1

尝试创建自定义授权属性，如this。

public class AuthorizeRolesAttribute : AuthorizeAttribute
{
    public AuthorizeRolesAttribute(params string[] roles) : base()
    {
        Roles = string.Join(",", roles);
    }
}

假设多个控制器的角色相同，创建一个辅助类：

public static class Role
{
    public const string Administrator = "Administrator";
    public const string Assistant = "Assistant";
}

然后像这样使用它：

public class MyController : Controller
{
    [AuthorizeRoles(Role.Administrator, Role.Assistant)]
    public ActionResult AdminOrAssistant()
    {                       
        return View();
    }
}

Answer 2

确保您从System.Web.Mvc.AuthorizeAttribute 而非System.Web.Http.AuthorizeAttribute 派生自定义属性类。

我遇到了同样的问题。一旦我改变它，一切正常。

您可能还想将以下内容添加到您的自定义属性类中：

[AttributeUsage(AttributeTargets.Method | AttributeTargets.Class, Inherited = true, AllowMultiple = true)] 

Answer 3

我发现解决此问题的最佳和最简单的方法就是在 Authorize 属性中连接角色。

[Authorize(Roles = CustomRoles.Admin + "," + CustomRoles.OtherRole)]

使用 CustomRole 的类具有像这样的常量字符串：

public static class CustomRoles
{
    public const string Admin = "Admin";
    // and so on..
}

Answer 4

我觉得自定义授权属性对于这个问题来说是多余的，除非你有大量的角色。

既然字符串必须在编译时知道，为什么不创建一个静态 Role 类，其中包含您已定义的角色的公共字符串，然后添加逗号分隔的字符串与您想要授权的某些角色：

public static class Roles
{
    public const string ADMIN = "Admin";
    public const string VIEWER = "Viewer";

    public const string ADMIN_OR_VIEWER = ADMIN + "," + VIEWER;
}

然后你可以在控制器类或控制器方法（或两者）上使用授权属性：

[Authorize(Roles = Roles.ADMIN]
public class ExampleController : Controller
{
    [Authorize(Roles = Roles.ADMIN_OR_VIEWER)
    public ActionResult Create()
    {
        ..code here...
    }
}

Answer 5

我所做的是@Tieson 中的答案

我对他的回答稍作调整。而不是 string.Join 为什么不将其转换为列表？

这是我的答案：

public class AuthorizeRolesAttribute : AuthorizeAttribute
{
    private new List<string> Roles;
    public AuthorizeRolesAttribute(params string[] roles) : base()
    {
        Roles = roles.toList()
    }
}

然后检查角色是否有效覆盖 OnAuthorization

public override void OnAuthorization(HttpActionContext actionContext)
{
            if (Roles == null)
                HandleUnauthorizedRequest(actionContext);
            else
            {
                ClaimsIdentity claimsIdentity = HttpContext.Current.User.Identity as ClaimsIdentity;
                string _role = claimsIdentity.FindFirst(ClaimTypes.Role).Value;
                bool isAuthorize = Roles.Any(role => role == _role);

                if(!isAuthorize)
                    HandleUnauthorizedRequest(actionContext);
            }
        }

你有它，它现在正在验证角色是否被授权访问资源