Nic*_*olk 5 c# asp.net asp.net-mvc entity-framework asp.net-identity
我有以下注册方法,我发誓我(手动)测试了这个方法一段时间,并注意到如果用户名已经存在,则结果只是 result.Succeeded 的错误值,并将错误消息附加到 ModelState (使用AddErrors(result) 辅助方法中的构建)。我非常确定此方法 ( Register(...)) 是 ASP.NET mvc 5 中开箱即用的,但我认为我更改了用户以包含用户名(而开箱即用时,电子邮件仅用作用户名)。
public async Task<ActionResult> Register(RegisterViewModel model)
{
if (ModelState.IsValid)
{
var user = new ApplicationUser { UserName = model.Username, Email = model.Email };
var result = await UserManager.CreateAsync(user, model.Password);
if (result.Succeeded)
{
await SignInManager.SignInAsync(user, isPersistent: false, rememberBrowser: false);
return RedirectToAction("Index", "Home");
}
AddErrors(result);
}
// If we got this far, something failed, redisplay form
return View(model);
}
Run Code Online (Sandbox Code Playgroud)
相反,我目前收到的错误是 EntityValidationError 被抛出且未被捕获。
我知道我可以简单地捕获此错误并继续我的一天,但我想确保如果不是正确的行为,是否是其他原因导致了此问题。
更新:
创建新的 MVC 项目后,我可以确认典型的行为(注册重复的用户名时)是 CreateAsync 应返回结果为 false 值的结果。成功,并应附加“用户名已被采用”的错误消息到模型状态。显然我的代码或配置有问题,但我不知道从哪里开始探索。如果有帮助的话,我最近在代码的其他地方看到了 EntityValidationErrors,而这些情况也不应该保证。请参阅:无法在数据库更新时保存更改。奇怪的延迟加载行为可能吗?
我找到了自己的解决方案。正如我所提到的,我已更改用户以包含用户名(并使电子邮件可选)。此任务的一部分涉及创建自定义用户验证器类。在自定义用户验证器的 ValidateAsync 方法中,我忘记检查用户名是否已存在(并且不属于该用户)。就像这样:
async Task<IdentityResult> IIdentityValidator<TUser>.ValidateAsync(TUser item)
{
var errors = new List<string>();
// ...
// Piece of code I have now added
var owner = await _manager.FindByNameAsync(item.UserName);
if (owner != null && !EqualityComparer<string>.Default.Equals(owner.Id, item.Id))
{
errors.Add($"Username {item.UserName} is already taken");
}
// End of code I added
// ...
return errors.Any()
? IdentityResult.Failed(errors.ToArray())
: IdentityResult.Success;
}
Run Code Online (Sandbox Code Playgroud)
我相信我学到的教训是应用程序层验证之间的区别,其中验证发生在 UserManager 的 CreateAsync 方法中。在应用层验证的情况下,错误将完全按照规定出现。如果省略该层验证,并且数据库面临相同的约束,那么当保存上下文时,它将抛出自己的错误。在本例中,出现了一个稍微神秘的 EntityValidationError。
| 归档时间: |
|
| 查看次数: |
2380 次 |
| 最近记录: |