3

我正在编写一个 MVC 3 应用程序,用户可以在其中登录并管理他们的数据。我想防止用户查看或篡改其他用户的数据。我的第一直觉是在每个动作方法中验证对相关对象的访问,如下所示:

public ActionResult ShowDetails(int objectId)
{
    DetailObject detail = _repo.GetById(objectId);
    if (detail.User.UserID != (Guid)Membership.GetUser().ProviderUserKey)
    {
        return RedirectToAction("LogOff", "Account");
    }
}

这很好用,但我认为将对象授权代码放入从 AuthorizeAttribute 派生的自定义 Authorize 属性中可能会更好,然后我可以将其应用于控制器。不幸的是,我无法从我的自定义 Authorize 属性中找到访问操作方法参数的方法。相反,我发现访问传入 objectId 的唯一方法是检查 httpContext.Request 或 filterContext.RequestContext.RouteData.Values:

public class MyAuthorizeAttribute : AuthorizeAttribute
{
    private int _objectId = 0;
    private IUnitOfWork _unitOfWork;

    public MyAuthorizeAttribute(IUnitOfWork uow)
    {
        _unitOfWork = uow;
    }

    public override void OnAuthorization(AuthorizationContext filterContext)
    {
        int.TryParse((string) filterContext.RequestContext.RouteData.Values["id"], out _objectId);
        base.OnAuthorization(filterContext);
    }

    protected override bool AuthorizeCore(HttpContextBase httpContext)
    {
        int objectId = 0;
        if (httpContext.Request.Params.AllKeys.Contains("id", StringComparer.InvariantCultureIgnoreCase))
        {
            int.TryParse(httpContext.Request[idKey], out objectId);
        }

        if (objectId != 0)
        {
            if (!IsAuthorized(objectId, httpContext.User.Identity.Name))
            {
                return false;
            }
        }

        if (_objectId != 0)
        {
            if (!IsAuthorized(objectId, httpContext.User.Identity.Name))
            {
                return false;
            }
        }

        return base.AuthorizeCore(httpContext);
    }

    private bool IsAuthorized(int objectId, string userName)
    {
        DetailObject detail;
        detail = _unitOfWork.ObjectRepository.GetById(objectId);

        if (detail == null)
        {
            return false;
        }

        if (userName != detail.User.UserName)
        {
            return false;
        }

        return true;
    }
}

我发现这种方法非常笨拙。我真的不想在 RouteData 或 Request 对象中闲逛;能够访问操作方法参数会更干净,因为模型绑定已经从 RouteData 和 Request 中提取了相关数据。

我知道我可以从自定义操作过滤器访问操作方法参数(详见此处),但数据授权代码不应该放在授权过滤器中吗?我看到的授权过滤器示例越多,我就越觉得它们仅用于处理角色。

我的主要问题是:如何从我的自定义授权属性访问操作方法参数?

4

1 回答 1

3

回答您的主要问题:不,遗憾AuthorizationContext的是不提供对操作参数的访问。

首先,您可以使用ValueProviderto 不必处理 id 是路由的一部分还是查询参数或发布的 HTTP,如下所示:

public override void OnAuthorization(AuthorizationContext filterContext)
{
    string id = filterContext.Controller.ValueProvider.GetValue("id").AttemptedValue;
    ...
}

这适用于简单的数据类型并且引入的开销很小。然而,一旦你开始为你的动作参数使用自定义模型绑定器,你必须继承你的过滤器ActionFilterAttribute以避免双重绑定:

[MyFilter]
public ActionResult MyAction([ModelBinder(typeof(MyModelBinder))] MyModel model)
{
    ...
}

public class MyFilterAttribute : ActionFilterAttribute
{
    public override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        var model = filterContext.ActionParameters["model"] as MyModel;
        ...
    }
}

虽然出于授权目的从语义上继承AuthorizeAttribute听起来更好,但没有其他理由这样做。此外,我发现使用ActionFilterAttribute起来更容易,因为您只需覆盖一个方法,而不是为后续方法保留状态。

于 2014-07-04T14:09:02.220 回答