Asp.net核心身份更改用户名/电子邮件

带有确认逻辑的默认身份更改用户名/电子邮件没有意义。

将应用程序设置为需要电子邮件确认
设置需要确认的电子邮件才能登录
然后用户更改电子邮件，错误输入电子邮件，注销
现在用户被锁定。电子邮件已更改，但需要确认登录，没有电子邮件确认链接，因为地址输入错误

是我的应用程序设置错误，还是Microsoft没有很好地设计Identity？

public async Task<IActionResult> OnPostAsync()
{
if (!ModelState.IsValid)
{
return Page();
}
var user = await _userManager.GetUserAsync(User);
if (user == null)
{
return NotFound($"Unable to load user with ID '{_userManager.GetUserId(User)}'.");
}
//...
var email = await _userManager.GetEmailAsync(user);
if (Input.Email != email)
{
var setEmailResult = await _userManager.SetEmailAsync(user, Input.Email);
if (!setEmailResult.Succeeded)
{
var userId = await _userManager.GetUserIdAsync(user);
throw new InvalidOperationException($"Unexpected error occurred setting email for user with ID '{userId}'.");
}
StatusMessage = "<strong>Verify your new email</strong><br/><br/>" +
"We sent an email to " + Input.Email +
" to verify your address. Please click the link in that email to continue.";
}
//...

await _signInManager.RefreshSignInAsync(user);
return RedirectToPage();
}

您的问题是为此目的使用SetEmailAsync。该方法旨在为当前不存在的用户设置电子邮件。在这种情况下，将confirmed设置为false是有意义的，不会造成任何问题。

还有另一种方法ChangeEmailAsync，这是您应该使用的方法。此方法需要一个令牌，该令牌将从电子邮件确认流中获得。换句话说，你应该采取的步骤是：

用户提交带有新电子邮件的表单以更改为
您将向用户发送一封确认电子邮件。用户要更改的电子邮件地址需要保留在确认链接中或数据库中的单独位置。换句话说，用户在其用户记录中的实际电子邮件没有更改
用户单击电子邮件中的确认链接。您可以从链接或您以前保存的任何位置获得他们想要更改的新电子邮件地址
您使用此电子邮件和来自确认链接的令牌致电ChangeEmailAsync
用户的电子邮件现已更改并确认

编辑

FWIW，是的，这似乎是默认模板的问题。不确定他们为什么这样做，因为是的，这在很大程度上打破了局面，正如我在回答中所说，ChangeEmailAsync的存在正是为了这个目的。只需按照我上面概述的步骤，在这里更改逻辑，以了解用户通过"管理"页面提交新电子邮件地址时会发生什么。

编辑#2

我已经为此在Github上提交了一个问题。我现在不能再花更多的时间了，但如果我有时间，而且没有人能打败我，我会尝试提交一个修复请求。修复相对来说是直接的。

编辑#3

我能够在一个分叉中获得一个基本的电子邮件更改流程。然而，团队已经分配了这个问题，并且似乎将其作为对Identity UI进行更大改革的一部分。我现在可能不会再花更多的时间了，但鼓励你关注这个问题，了解团队的最新情况。如果您现在确实借用了我的代码来实现修复，请注意，我正试图创建一个对其他代码熵最小的解决方案。例如，在真正的生产应用程序中，您应该将新电子邮件保存在数据库中的某个位置，而不是在URL中传递。

如前所述，模板肯定提供了错误的行为。您可以在https://github.com/aspnet/Scaffolding回购。

我建议在GitHub项目上提出一个问题，这样就可以改变了。当模板更新时，他们无疑必须考虑启用确认和未启用确认的情况。在您的情况下，您可以相对容易地重用OnPostSendVerificationEmailAsync()中已经存在的逻辑。

一个更通用的实现看起来像这样：

public partial class IndexModel : PageModel
{
// inject as IOptions<IdentityOptions> into constructor
private readonly IdentityOptions _options;
// Extracted from OnPostSendVerificationEmailAsync()
private async Task SendConfirmationEmail(IdentityUser user, string email)
{
var userId = await _userManager.GetUserIdAsync(user);
var code = await _userManager.GenerateEmailConfirmationTokenAsync(user);
var callbackUrl = Url.Page(
"/Account/ConfirmEmail",
pageHandler: null,
values: new { userId = userId, code = code },
protocol: Request.Scheme);
await _emailSender.SendEmailAsync(
email,
"Confirm your email",
$"Please confirm your account by <a href='{HtmlEncoder.Default.Encode(callbackUrl)}'>clicking here</a>.");
}
public async Task<IActionResult> OnPostAsync()
{
//... Existing code
var email = await _userManager.GetEmailAsync(user);
var confirmationEmailSent = false;
if (Input.Email != email)
{
if(_options.SignIn.RequireConfirmedEmail)
{
// new implementation
await SendConfirmationEmail(user, Input.Email);
confirmationEmailSent = true;
}
else
{
// current implementation
var setEmailResult = await _userManager.SetEmailAsync(user, Input.Email);
if (!setEmailResult.Succeeded)
{
var userId = await _userManager.GetUserIdAsync(user);
throw new InvalidOperationException($"Unexpected error occurred setting email for user with ID '{userId}'.");
}
}
var setEmailResult = await _userManager.SetEmailAsync(user, Input.Email);
if (!setEmailResult.Succeeded)
{
var userId = await _userManager.GetUserIdAsync(user);
throw new InvalidOperationException($"Unexpected error occurred setting email for user with ID '{userId}'.");
}
}
// existing update phone number code;
await _signInManager.RefreshSignInAsync(user);
StatusMessage = confirmationEmailSent 
? "Verification email sent. Please check your email."
: "Your profile has been updated";
return RedirectToPage();
}

public async Task<IActionResult> OnPostSendVerificationEmailAsync()
{
if (!ModelState.IsValid)
{
return Page();
}
var user = await _userManager.GetUserAsync(User);
if (user == null)
{
return NotFound($"Unable to load user with ID '{_userManager.GetUserId(User)}'.");
}
var email = await _userManager.GetEmailAsync(user);
await SendConfirmationEmail(user, email);
StatusMessage = "Verification email sent. Please check your email.";
return RedirectToPage();
}
}

相关内容

最新更新

热门标签：