有没有办法集中执行每个操作方法必须具有“ValidateAntiForgeryToken”属性?我认为必须通过扩展一个“路由”类来完成。
编辑或者在应用程序启动时做一些反思?
答案 0 :(得分:22)
是。您可以通过创建自己的继承Mvc控制器的BaseController并重载OnAuthorization()来实现此目的。您希望在执行之前确保它是POST事件:
public abstract class MyBaseController : Controller
{
protected override void OnAuthorization(AuthorizationContext filterContext)
{
//enforce anti-forgery stuff for HttpVerbs.Post
if (String.Compare(filterContext.HttpContext.Request.HttpMethod,
System.Net.WebRequestMethods.Http.Post, true) == 0)
{
var forgery = new ValidateAntiForgeryTokenAttribute();
forgery.OnAuthorization(filterContext);
}
base.OnAuthorization(filterContext);
}
}
完成后,确保所有控制器都继承自此MyBaseController(或您所称的任何控制器)。或者,如果您喜欢使用相同的代码,则可以在每个控制器上执行此操作。
答案 1 :(得分:8)
听起来你正试图阻止“哎呀我忘了设置那个”错误。如果是这样,我认为最好的地方是使用自定义ControllerActionInvoker。
基本上你想做的就是阻止MVC甚至找不到没有AntiForgery令牌的动作:
public class MustHaveAntiForgeryActionInvoker : ControllerActionInvoker
{
protected override ActionDescriptor FindAction(ControllerContext controllerContext, ControllerDescriptor controllerDescriptor, string actionName)
{
var foundAction = base.FindAction(controllerContext, controllerDescriptor, actionName);
if( foundAction.GetCustomAttributes(typeof(ValidateAntiForgeryTokenAttribute), true ).Length == 0 )
throw new InvalidOperationException("Can't find a secure action method to execute");
return foundAction;
}
}
然后在您的控制器中,最好是您的基本控制器:
ActionInvoker = new MustHaveAntiForgeryActionInvoker();
只是想添加自定义Controller基类往往变得“厚”,并且使用MVC卓越的可扩展性点来连接它们所需的功能时,它总是最佳实践。
这是MVC大多数可扩展性点的一个很好的指南: http://codeclimber.net.nz/archive/2009/04/08/13-asp.net-mvc-extensibility-points-you-have-to-know.aspx
答案 2 :(得分:7)
好的,我刚刚将项目升级到MVC v2.0,如果您在控制器操作中使用AuthorizeAttribute
,eduncan911的解决方案将不再起作用。有点难以弄明白为什么。
因此,故事中的罪魁祸首是MVC团队在ViewContext.HttpContext.User.Identity.Name
的值中添加了RequestVerificationToken
属性的使用。
基本控制器中被覆盖的OnAuthorization
在控制器操作上的任何过滤器之前执行。因此,问题是尚未调用Authorize属性,因此未设置ViewContext.HttpContext.User
。所以UserName是String.Empty,而用于验证的AntiForgeryToken包括真实用户名=失败。
我们现在用以下代码解决了这个问题:
public abstract class MyBaseController : Controller
{
protected override void OnAuthorization(AuthorizationContext filterContext)
{
//enforce anti-forgery stuff for HttpVerbs.Post
if (String.Compare(filterContext.HttpContext.Request.HttpMethod, "post", true) == 0)
{
var authorize = new AuthorizeAttribute();
authorize.OnAuthorization(filterContext);
if (filterContext.Result != null) // Short circuit validation
return;
var forgery = new ValidateAntiForgeryTokenAttribute();
forgery.OnAuthorization(filterContext);
}
base.OnAuthorization(filterContext);
}
}
对MVC代码库的一些引用:
ControllerActionInvoker#InvokeAuthorizationFilters()
第283行。同样短路。
AntiForgeryData#GetUsername()
第98行。新功能。
答案 3 :(得分:1)
这个怎么样?
[ValidateAntiForgeryToken]
public class MyBaseController : Controller
{
}