ASP.NET Identity 验证 ResetPassword 令牌是否已过期

Mis*_*siu 7 c# asp.net asp.net-web-api asp.net-identity-2

在我的 API 中,我有 2 个端点,首先生成电子邮件以重置密码表单(我使用 生成令牌UserManager.GeneratePasswordResetTokenAsync)。
第二个端点用于实际密码重置(我使用UserManager.ResetPasswordAsync)。

我的要求是验证密码重置所需的令牌是否未过期。

在 GitHub 上搜索我发现了这个问题,并且从设计上我发现这是不可能的。

然而,更深入地搜索我发现UserManager.ResetPasswordAsync 在内部使用ValidateAsync来自Microsoft.AspNet.Identity.Owin.DataProtectorTokenProvider

有了这个,我创建了这个扩展方法:

using Microsoft.AspNet.Identity;
using Microsoft.AspNet.Identity.Owin;
using System;
using System.Globalization;
using System.IO;
using System.Text;

namespace Api.Extensions
{
    public enum TokenValidity
    {
        VALID,
        INVALID,
        INVALID_EXPIRED,
        ERROR
    }

    public static class UserManagerExtensions
    {
        public static TokenValidity IsResetPasswordTokenValid<TUser, TKey>(this UserManager<TUser, TKey> manager, TUser user, string token) where TKey : IEquatable<TKey> where TUser : class, IUser<TKey>
        {
            return IsTokenValid(manager, user, "ResetPassword", token);
        }

        public static TokenValidity IsTokenValid<TUser, TKey>(this UserManager<TUser, TKey> manager, TUser user, string purpose, string token) where TKey : IEquatable<TKey> where TUser : class, IUser<TKey>
        {
            try
            {
                //not sure if this is needed??
                if (!(manager.UserTokenProvider is DataProtectorTokenProvider<TUser, TKey> tokenProvider)) return TokenValidity.ERROR;

                var unprotectedData = tokenProvider.Protector.Unprotect(Convert.FromBase64String(token));
                var ms = new MemoryStream(unprotectedData);
                using (var reader = ms.CreateReader())
                {
                    var creationTime = reader.ReadDateTimeOffset();
                    var expirationTime = creationTime + tokenProvider.TokenLifespan;

                    var userId = reader.ReadString();
                    if (!String.Equals(userId, Convert.ToString(user.Id, CultureInfo.InvariantCulture)))
                    {
                        return TokenValidity.INVALID;
                    }

                    var purp = reader.ReadString();
                    if (!String.Equals(purp, purpose))
                    {
                        return TokenValidity.INVALID;
                    }

                    var stamp = reader.ReadString();
                    if (reader.PeekChar() != -1)
                    {
                        return TokenValidity.INVALID;
                    }

                    var expectedStamp = "";
                    //if supported get security stamp for user
                    if (manager.SupportsUserSecurityStamp)
                    {
                        expectedStamp = manager.GetSecurityStamp(user.Id);
                    }

                    if (!String.Equals(stamp, expectedStamp)) return TokenValidity.INVALID;

                    if (expirationTime < DateTimeOffset.UtcNow)
                    {
                        return TokenValidity.INVALID_EXPIRED;
                    }

                    return TokenValidity.VALID;
                }
            }
            catch
            {
                // Do not leak exception
            }
            return TokenValidity.INVALID;
        }
    }

    internal static class StreamExtensions
    {
        internal static readonly Encoding DefaultEncoding = new UTF8Encoding(false, true);

        public static BinaryReader CreateReader(this Stream stream)
        {
            return new BinaryReader(stream, DefaultEncoding, true);
        }

        public static BinaryWriter CreateWriter(this Stream stream)
        {
            return new BinaryWriter(stream, DefaultEncoding, true);
        }

        public static DateTimeOffset ReadDateTimeOffset(this BinaryReader reader)
        {
            return new DateTimeOffset(reader.ReadInt64(), TimeSpan.Zero);
        }

        public static void Write(this BinaryWriter writer, DateTimeOffset value)
        {
            writer.Write(value.UtcTicks);
        }
    }
}
Run Code Online (Sandbox Code Playgroud)

所以现在我可以添加这个检查:

if (UserManager.IsResetPasswordTokenValid(user, model.Code) == TokenValidity.INVALID_EXPIRED)
{
    return this.BadRequest("errorResetingPassword", "Link expired");
}
Run Code Online (Sandbox Code Playgroud)

我的问题是:

1.有没有更简单的方法来做到这一点?
我的目的是向用户显示电子邮件中的链接已过期的信息,因为现在他只能看到重置密码有问题。

2.如果没有内置的方法来做这件事,潜在的安全漏洞是什么?我使用我的扩展方法作为额外的检查。如果我的方法返回 true,我仍然使用ResetPasswordAsync.

小智 5

UserManager 具有您可以使用的 VerifyUserTokenAsync 和 VerifyUserToken 方法。

请参阅Wouter 对“如何检查密码重置令牌是否已过期?”问题的回答。更多细节。

所以你可以使用类似的东西

if (!UserManager.VerifyUserToken(userId, "ResetPassword", model.code)){
  return this.BadRequest("errorResetingPassword", "Link expired");
}
Run Code Online (Sandbox Code Playgroud)

  • 我已经尝试过了,但问题是它会检查所有内容:创建日期、用户 ID、用途和安全标记。调用 `VerifyUserToken` 之后你会知道 token 是无效的,但你不会知道为什么它无效。我想明确知道 userId 有效,目的有效,安全标记有效但令牌已过期。希望这是有道理的:) (2认同)