在我的asp.net mvc应用程序中,当用户注册时,我会向他们的帐户发送一封电子邮件,其中包含用于验证的链接,然后才能使用该应用。请参阅下面的代码段。
var emailActionLink = Url.Action("ValidateAccount", "Register",
new { Token = registeredUserViewModel.Id, Username = registeredUserViewModel.Username },
Request.Url.Scheme);
上面的代码段是他们点击的代码,然后会调用路线值
的动作验证帐户操作
public ActionResult ValidateAccount(string token, string username)
{
try
{
if (!string.IsNullOrEmpty(token) && !string.IsNullOrEmpty(username))
{
var user = _userServiceClient.IsUserNameAvailable(username);
if (!user.HasValue) throw new NullReferenceException("This account does not exist");
var userContract = user.Value;
userContract.EmailVerified = true;
if (_userServiceClient.UpdateUser(userContract) == null) throw new Exception("Something has gone wrong");
return View("ValidationCompleted");
}
else
{
ViewBag.RegisteredUser = null;
}
}
catch (Exception exception)
{
throw;
}
return View();
}
问题是,这种方法不会验证令牌,如果某人更改了uri中token
的值,会发生什么情况,这仍然会通过,而且帐户也是如此。改善这一点的正确方法是什么。
在这种情况下,令牌是用户的ID,它是Guid,但它是经过编码的,无法将我的数据库中的用户ID与此编码令牌进行比较。我认为这是在动作链接中编码的。
答案 0 :(得分:1)
不是使用你的Id,你可能最好在表中使用Token字段(可以为空,以便在验证后将其清除)。生成一个URL安全令牌(我使用十六进制字符串,它不使用任何特殊字符),然后在验证操作中在数据库中查找该令牌。
以下是令牌生成器的示例:
public class TokenGenerator
{
public static string GenerateToken(int size = 32)
{
var crypto = new RNGCryptoServiceProvider();
byte[] rbytes = new byte[size / 2];
crypto.GetNonZeroBytes(rbytes);
return ToHexString(rbytes, true);
}
private static string ToHexString(byte[] bytes, bool useLowerCase = false)
{
var hex = string.Concat(bytes.Select(b => b.ToString(useLowerCase ? "x2" : "X2")));
return hex;
}
}
然后,将适当的方法添加到您的服务类:
public YourUserType GetUserForToken(string token, string userName)
{
return YourDbContext.Users
.SingleOrDfault(user => user.Token.Equals(token, StringComparison.OrdinalIgnoreCase)
&& user.UserName.Equals(userName, StringComparison.OrdinalIgnoreCase));
}
显然,这会对您的表结构和数据访问代码做出一些假设。