8

我计划将下面的代码用于我的 web api 安全性，但我不确定这是否足够安全和​​合乎逻辑的方式。我不想使用 OWIN 和 AspNet.Identity，因为它对我来说非常复杂，而且我不完全理解，我不知道如何自定义数据库表、用户角色等。但我的方式很简单，而且非常可定制我。

这是 CustomAuthorizeAttribute；

public class CustomAuthorize : AuthorizeAttribute
{
    public override void OnAuthorization(HttpActionContext actionContext)
    {
        if ((actionContext.Request.Headers.GetValues("Host").FirstOrDefault().Contains("localhost:15742")))
        {
            IEnumerable<string> access_token;
            if (actionContext.Request.Headers.TryGetValues("Authorization", out access_token))
            {
                var user = GetUserByToken(access_token);
                if (user!=null && !user.TokenIsExpired)
                {
                    HttpContext.Current.Response.AddHeader("WWW-Authenticate", "Custom " + access_token.FirstOrDefault());
                    return;
                }
                else
                {
                    actionContext.Response = actionContext.Request.CreateResponse(HttpStatusCode.Unauthorized);
                    HttpContext.Current.Response.AddHeader("WWW-Authenticate", "Custom");
                    return;
                }
            }
            else
            {
                actionContext.Response = actionContext.Request.CreateResponse(HttpStatusCode.Unauthorized);
            }
        }
        else
        {
            actionContext.Response = actionContext.Request.CreateResponse(HttpStatusCode.Forbidden);
        }
    }
}

这是前端使用

<script type="text/javascript">
    $(function () {
        var access_token = $.cookie('access_token');
        if (access_token == undefined) {
            $.cookie('access_token', 'test-token');
        }


        $.ajax({
            url: '/api/account',
            headers: { access_token: access_token },
            success: function (data) {
                document.write(data.name + " " + data.lastname);
            }
        });
    });
</script>

顺便说一句，我为我的英语感到抱歉。我希望你能理解我的问题，我正在等待你的建议。

4

1 回答 1

0

Necroreply 对于那些希望制作自定义身份验证属性的人:)

第一个检查是多余的，因为 HTTP 请求只是 TCP 连接上的一串文本，因此任何人都可以使用 TCP 客户端连接到您的服务器并发送他想要的任何标头。 actionContext.Request.Headers.GetValues("Host").FirstOrDefault().Contains("localhost:15742"))

根据https://docs.microsoft.com/en-us/aspnet/web-api/overview/security/authentication-and-authorization-in-aspnet-web-api

授权过滤器在控制器操作之前运行。如果请求未被授权，过滤器将返回错误响应，并且不会调用操作。

您属性不设置响应的唯一方法是user!=null && !user.TokenIsExpired，此属性将完成工作并且可以被认为是安全的。

这个标题可以去掉HttpContext.Current.Response.AddHeader("WWW-Authenticate", "Custom");

另外，如果成功，为什么还要再次发送身份验证令牌？HttpContext.Current.Response.AddHeader("WWW-Authenticate", "Custom " + access_token.FirstOrDefault());

只需降低 IF-s 嵌套级别，以便代码更易于阅读：

    public override void OnAuthorization(HttpActionContext actionContext)
    {
        IEnumerable<string> access_token;

        if (!actionContext.Request.Headers.TryGetValues("Authorization", out access_token))
        {
            actionContext.Response = actionContext.Request.CreateResponse(HttpStatusCode.Unauthorized);
            return;
        }

        var user = GetUserByToken(access_token);

        if (user == null || user.TokenIsExpired)
        {
            actionContext.Response = actionContext.Request.CreateResponse(HttpStatusCode.Unauthorized);                
            return;
        }

        // OK
        return;
    }

许多 ASP.NET 代码似乎过度设计（有时是 :)，包括 OWIN。但它有一个目的——声明做各种事情的标准方式，例如身份验证。

想象一下，每个人都将开始构建他们的自定义属性，那么就不可能只安装 Google nuget 包并执行类似的操作

public void ConfigureAuth(IAppBuilder app)
{
     app.UseGoogleAuthentication(
         clientId: "000-000.apps.googleusercontent.com",
         clientSecret: "00000000000");
}
于 2017-08-01T09:17:04.640 回答