kocoded/oqtane.framework
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

log any user creation errors from .NET Identity

Browse Source
This commit is contained in:
Shaun Walker 2022-09-29 17:16:29 -04:00
parent ddf1caaaaa
commit e01c3e7e4a
1 changed files with 14 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
Oqtane.Server/Controllers/UserController.cs
View File

@ -165,6 +165,7 @@ namespace Oqtane.Controllers
if (allowregistration) if (allowregistration)
{ {
bool succeeded; bool succeeded;
string errors = "";
IdentityUser identityuser = await _identityUserManager.FindByNameAsync(user.Username); IdentityUser identityuser = await _identityUserManager.FindByNameAsync(user.Username);
if (identityuser == null) if (identityuser == null)
{ {
@ -174,12 +175,20 @@ namespace Oqtane.Controllers
identityuser.EmailConfirmed = verified; identityuser.EmailConfirmed = verified;
var result = await _identityUserManager.CreateAsync(identityuser, user.Password); var result = await _identityUserManager.CreateAsync(identityuser, user.Password);
succeeded = result.Succeeded; succeeded = result.Succeeded;
if (!succeeded)
{
errors = string.Join(", ", result.Errors.Select(e => e.Description));
}
} }
else else
{ {
var result = await _identitySignInManager.CheckPasswordSignInAsync(identityuser, user.Password, false); var result = await _identitySignInManager.CheckPasswordSignInAsync(identityuser, user.Password, false);
succeeded = result.Succeeded; succeeded = result.Succeeded;
verified = true; if (!succeeded)
{
errors = "Password Not Valid For User";
}
verified = succeeded;
} }
if (succeeded) if (succeeded)
@ -188,6 +197,10 @@ namespace Oqtane.Controllers
user.LastIPAddress = ""; user.LastIPAddress = "";
newUser = _users.AddUser(user); newUser = _users.AddUser(user);
} }
else
{
_logger.Log(user.SiteId, LogLevel.Error, this, LogFunction.Create, "Unable To Add User {Username} - {Errors}", user.Username, errors);
}
if (newUser != null) if (newUser != null)
{ {