ASP .NET MVC: Redirect to a view if certain condition is not met

Question

I finally got to try .NET MVC this weekend and, as was expected, ran into some hurdles.

I am trying to password-protect my site while it's under development. The basic idea is very very simple: in my BaseController class (that inherits from the Controller and is inherited by all other controller classes) I check whether a certain Session variable is set. If it is not set, I need to redirect them to a specific view, where users can enter a password. The key is that this check should happen on every request, regardless of what part of the site they're accessing. This is what I tried but has failed: Override OnActionExecuting in the BaseController class and check for the following:

if ((Session["siteAuth"] == null || (bool)Session["siteAuth"] != true) && 
        filterContext.ActionDescriptor.ActionName != "EnterPassword") {
   RedirectToAction("EnterPassword");
}
else {
   base.OnActionExecuting(filterContext);
}

OnActionExecuting executes on every request, however, RedirectToAction does not happen. Basically it just renders the view that it would otherwise render.

The closest solution that I found was here, but mine needs to happen on every request.

Any hints as to what I'm doing wrong?

Daniel Liuzzi · Accepted Answer · 2009-11-22 20:40:12Z

12

Why don't you just decorate your action or controller with [Authorize]?

answered Nov 22, 2009 at 20:40

Daniel Liuzzi

17.3k8 gold badges55 silver badges57 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

MK_Dev Over a year ago

Perhaps because I had no idea such thing existed. Researching it right now. Thanks!

LukLed · Accepted Answer · 2009-11-22 20:47:14Z

4

You should set filterContext.Result:

    protected override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        filterContext.Result = RedirectToAction("EnterPassword");
    }

but as Daniel Liuzzi said, there are other better ways to do it.

answered Nov 22, 2009 at 20:47

LukLed

31.9k17 gold badges87 silver badges108 bronze badges

1 Comment

iCollect.it Ltd Over a year ago

+1 this may not have been the best answer for the question here, but it was exactly what I needed for something more complex than a simple Authorize. Cheers.

Collectives™ on Stack Overflow

ASP .NET MVC: Redirect to a view if certain condition is not met

2 Answers 2

1 Comment

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related