【发布时间】:2022-04-28 16:38:32
【问题描述】:
我正在学习使用 .NET Identity。我必须对 Active Directory 进行身份验证。为此,我正在尝试使用ActiveDirecotoryMembershipProvider。我必须:
- 根据Active Directory (AD) 验证用户/密码。
- 检查用户是否存在于我自己的数据库中。
我在web.config 中将其配置为使用ActiveDirectoryMembershipProvider 作为默认会员提供程序。然后我在我的ApplicationSignInManager 类(继承SignInManager)中覆盖了PasswordSignInAsync 方法,如下所示-
public override Task<SignInStatus> PasswordSignInAsync(string userName, string password, bool isPersistent, bool shouldLockout)
{
var adok = Membership.Provider.ValidateUser(userName, password);
if (adok)
{
var user = UserManager.FindByName(userName);
if (user == null)
return Task.FromResult<SignInStatus>(SignInStatus.Failure);
else
{
base.SignInAsync(user, isPersistent, shouldLockout);
return Task.FromResult<SignInStatus>(SignInStatus.Success);
}
}
else
return Task.FromResult<SignInStatus>(SignInStatus.Failure);
}
这似乎有效。但我认为这不是正确的做法。有没有更好的方法来实现这一点?
我是这样称呼上面的:
var result = await SignInManager.PasswordSignInAsync(username, password, isPersistent: false, shouldLockout: false);
switch (result)
{
case SignInStatus.Success:
return RedirectToAction("Index", "Home");
case SignInStatus.LockedOut:
return View("Lockout");
case SignInStatus.Failure:
default:
ModelState.AddModelError("", "Invalid login attempt.");
return View();
}
根据我得到的答案,我不应该在PasswordSignInAsync 中调用Membership Validate 方法。我同意这一点。事实上,我认为重写方法也是错误的。
还有人建议我使用UserLogins 为我的AD 提供一个提供者ID。但我能想到的唯一方法如下 -
IList<UserLoginInfo> loginInfos = await SignInManager.UserManager.GetLoginsAsync(username);
var valid = false;
foreach(var info in loginInfos)
{
valid = Membership.Providers[info.ProviderKey].ValidateUser(username, password);
if (valid)
break;
}
因此,如果我想针对多个 Providers 对用户进行身份验证,我可以为它们中的每一个创建一个提供程序密钥并将这些提供程序密钥分配给用户。这段代码将针对他们验证用户。但是我应该把这段代码放在哪里?我应该遵循什么约定?
我并不热衷于自己编写 AD 验证代码,因为我认为 ActiveDirectoryMembershipProvider 可以比我自己的代码做得更好。同样对于这两种情况,无论如何我都必须添加对System.DirectoryServices 的引用。
【问题讨论】:
-
不要只是投反对票,请发表评论,以便我了解您投反对票的原因。
-
我认为您可能被否决了,因为您的帖子似乎没有明显的问题。你有有效的代码,但想要其他途径的建议。这将更适合网站的代码审查部分:codereview.stackexchange.com
-
为什么?我已经清楚地说明了我想要的问题。我可能有有效的代码,但这并不意味着它是正确的。
-
@ozhug,我见过这个。它使用 Windows 身份验证。但正如我在问题中提到的那样,我需要两个级别的表单身份验证。
标签: c# asp.net asp.net-membership asp.net-identity asp.net-identity-2