ActionFilterAttribute для отключения SSL на контроллере Asp.Net MVC2 не работает согласованно

Question

Этот фильтр действий не работает согласованно. Иногда он отключает SSL, а иногда нет. У меня он применяется ко всему контроллеру в его объявлении.

   public class SSLFilter:ActionFilterAttribute
    {
            public override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        HttpRequestBase req = filterContext.HttpContext.Request;
        HttpResponseBase res = filterContext.HttpContext.Response;

        if (req.IsSecureConnection)
        {
            var builder = new UriBuilder(req.Url)
            {
                Scheme = Uri.UriSchemeHttp,
                Port = 80
            };
            res.Redirect(builder.Uri.ToString());
        }
        base.OnActionExecuting(filterContext);
    }
    }

Это немного странно ... есть идеи, почему это может работать спорадически?

Answer 1

Вы пробовали украшать свои контроллеры / действия атрибутом [RequireHttps]?

Ой, не заметил, что вы спрашивали о ASP.NET MVC 2. Этот атрибут доступен только в ASP.NET MVC 3, поэтому вот его исходный код (как реализовано в ASP.NET MVC 3):

using System;
using System.Diagnostics.CodeAnalysis;
using System.Web.Mvc.Resources;

[AttributeUsage(AttributeTargets.Class | AttributeTargets.Method, Inherited = true, AllowMultiple = false)]
public class RequireHttpsAttribute : FilterAttribute, IAuthorizationFilter {

    public virtual void OnAuthorization(AuthorizationContext filterContext) {
        if (filterContext == null) {
            throw new ArgumentNullException("filterContext");
        }

        if (!filterContext.HttpContext.Request.IsSecureConnection) {
            HandleNonHttpsRequest(filterContext);
        }
    }

    protected virtual void HandleNonHttpsRequest(AuthorizationContext filterContext) {
        // only redirect for GET requests, otherwise the browser might not propagate the verb and request
        // body correctly.

        if (!String.Equals(filterContext.HttpContext.Request.HttpMethod, "GET", StringComparison.OrdinalIgnoreCase)) {
            throw new InvalidOperationException(MvcResources.RequireHttpsAttribute_MustUseSsl);
        }

        // redirect to HTTPS version of page
        string url = "https://" + filterContext.HttpContext.Request.Url.Host + filterContext.HttpContext.Request.RawUrl;
        filterContext.Result = new RedirectResult(url);
    }

}

Обратите внимание, что вместо того, чтобы делать какие-либо перенаправления, он использует RedirectResult, который является правильным способом выполнения перенаправлений в ASP.NET MVC => путем возврата результатов действия:

filterContext.Result = new RedirectResult(url);

Мало того, что это будет выполнять правильное перенаправление, но и как закорачивать выполнение действия. Также семантически ваш фильтр должен быть IAuthorizationFilter , поскольку вы блокируете доступ к некоторому ресурсу здесь.