Пользовательская идентификация с использованием MVC5 и OWIN
Я пытаюсь добавить пользовательские свойства в ApplicationUser для веб-сайта с использованием аутентификации MVC5 и OWIN. Я прочитал /questions/8247303/aspnet-mvc-ustanovit-polzovatelskij-iidentity-ili-iprincipal/8247340#8247340 и мне нравится, как он интегрируется с базовым контроллером для быстрого доступа к новым свойствам. Моя проблема заключается в том, что когда я устанавливаю свойство HTTPContext.Current.User в свой новый IPrincipal, я получаю нулевую ссылку:
[NullReferenceException: Object reference not set to an instance of an object.]
System.Web.Security.UrlAuthorizationModule.OnEnter(Object source, EventArgs eventArgs) +127
System.Web.SyncEventExecutionStep.System.Web.HttpApplication.IExecutionStep.Execute() +136
System.Web.HttpApplication.ExecuteStep(IExecutionStep step, Boolean& completedSynchronously) +69
Вот мой код:
protected void Application_PostAuthenticateRequest(Object sender, EventArgs e)
{
if (HttpContext.Current.User.Identity.IsAuthenticated)
{
userManager = new UserManager<ApplicationUser>(new UserStore<ApplicationUser>(new ApplicationDbContext()));
ApplicationUser user = userManager.FindByName(HttpContext.Current.User.Identity.Name);
PatientPortalPrincipal newUser = new PatientPortalPrincipal();
newUser.BirthDate = user.BirthDate;
newUser.InvitationCode = user.InvitationCode;
newUser.PatientNumber = user.PatientNumber;
//Claim cPatient = new Claim(typeof(PatientPortalPrincipal).ToString(), );
HttpContext.Current.User = newUser;
}
}
public class PatientPortalPrincipal : ClaimsPrincipal, IPatientPortalPrincipal
{
public PatientPortalPrincipal(ApplicationUser user)
{
Identity = new GenericIdentity(user.UserName);
BirthDate = user.BirthDate;
InvitationCode = user.InvitationCode;
}
public PatientPortalPrincipal() { }
public new bool IsInRole(string role)
{
if(!string.IsNullOrWhiteSpace(role))
return Role.ToString().Equals(role);
return false;
}
public new IIdentity Identity { get; private set; }
public WindowsBuiltInRole Role { get; set; }
public DateTime BirthDate { get; set; }
public string InvitationCode { get; set; }
public string PatientNumber { get; set; }
}
public interface IPatientPortalPrincipal : IPrincipal
{
WindowsBuiltInRole Role { get; set; }
DateTime BirthDate { get; set; }
string InvitationCode { get; set; }
string PatientNumber { get; set; }
}
Я не нашел много документации о том, как это сделать, я прочитал эти статьи:
Комментарии во второй ссылке указывали на то, что я, возможно, использовал претензии ( http://msdn.microsoft.com/en-us/library/ms734687.aspx?cs-save-lang=1&cs-lang=csharp), но статья ссылалась чтобы не показать, как добавить их в IPrincipal (который является то, что HttpContext.Current.User есть), или где в конвейере вы должны добавить их в ClaimsIdentity (который является конкретным классом User является). Я склоняюсь к использованию утверждений, но мне нужно знать, где добавить эти новые утверждения пользователю.
Даже если претензии являются подходящим способом, мне любопытно, что я делаю неправильно с моим пользовательским IPrincipal, так как я, кажется, реализовал все, что требуется.
4 ответа
Я могу заставить что-то работать, используя Claims основанный на безопасности, так что если вы хотите сделать что-то быстро, вот что у меня есть на данный момент:
В процессе входа в систему AccountController (мой внутри SignInAsync метод), добавьте новую заявку к личности, созданной UserManager:
private async Task SignInAsync(ApplicationUser user, bool isPersistent)
{
AuthenticationManager.SignOut(DefaultAuthenticationTypes.ExternalCookie);
var identity = await UserManager.CreateIdentityAsync(user, DefaultAuthenticationTypes.ApplicationCookie);
identity.AddClaim(new Claim("PatientNumber", user.PatientNumber)); //This is what I added
AuthenticationManager.SignIn(new AuthenticationProperties() { IsPersistent = isPersistent }, identity);
}
Затем в моих базовых классах контроллеров я просто добавил свойство:
private string _patientNumber;
public string PatientNumber
{
get
{
if (string.IsNullOrWhiteSpace(_patientNumber))
{
try
{
var cp = ClaimsPrincipal.Current.Identities.First();
var patientNumber = cp.Claims.First(c => c.Type == "PatientNumber").Value;
_patientNumber = patientNumber;
}
catch (Exception)
{
}
}
return _patientNumber;
}
}
Эта ссылка была полезна для проверки требований: http://msdn.microsoft.com/en-us/library/ms734687.aspx?cs-save-lang=1&cs-lang=csharp
Обновление для проблемы с IPrincipal
Я отследил это до Identity имущество. Проблема заключалась в том, что я предоставлял конструктор по умолчанию на PatientPortalPrincipal класс, который не устанавливал свойство Identity. В итоге я удалил конструктор по умолчанию и вызвал правильный конструктор изнутри Application_PostAuthenticateRequestобновленный код ниже
protected void Application_PostAuthenticateRequest(Object sender, EventArgs e)
{
if (HttpContext.Current.User.Identity.IsAuthenticated)
{
userManager = new UserManager<ApplicationUser>(new UserStore<ApplicationUser>(new ApplicationDbContext()));
ApplicationUser user = userManager.FindByName(HttpContext.Current.User.Identity.Name);
PatientPortalPrincipal newUser = new PatientPortalPrincipal(user);
newUser.BirthDate = user.BirthDate;
newUser.InvitationCode = user.InvitationCode;
newUser.PatientNumber = user.PatientNumber;
//Claim cPatient = new Claim(typeof(PatientPortalPrincipal).ToString(), );
HttpContext.Current.User = newUser;
}
}
Это заставляет все это работать!
Вы получаете исключение, потому что HttpContext.Current.User.Identity.IsAuthenticated возвращает false в точке проверки (так же HttpContext.Current.Request.IsAuthenticated).
Если вы удалите if (HttpContext.Current.User.Identity.IsAuthenticated) Скажите, что это будет работать нормально (по крайней мере, эта часть кода).
Я пробовал простую вещь, как это:
BaseController.cs
public abstract class BaseController : Controller
{
protected virtual new CustomPrincipal User
{
get { return HttpContext.User as CustomPrincipal; }
}
}
CustomPrincipal.cs
public class CustomPrincipal : IPrincipal
{
public IIdentity Identity { get; private set; }
public bool IsInRole(string role) { return false; }
public CustomPrincipal(string username)
{
this.Identity = new GenericIdentity(username);
}
public DateTime BirthDate { get; set; }
public string InvitationCode { get; set; }
public int PatientNumber { get; set; }
}
Global.asax.cs
protected void Application_PostAuthenticateRequest(Object sender, EventArgs e)
{
CustomPrincipal customUser = new CustomPrincipal(User.Identity.Name);
customUser.BirthDate = DateTime.Now;
customUser.InvitationCode = "1234567890A";
customUser.PatientNumber = 100;
HttpContext.Current.User = customUser;
}
HomeController.cs
public ActionResult Index()
{
ViewBag.BirthDate = User.BirthDate;
ViewBag.InvitationCode = User.InvitationCode;
ViewBag.PatientNumber = User.PatientNumber;
return View();
}
И это работает нормально. Так что, если этот код:
userManager = new UserManager<ApplicationUser>(new UserStore<ApplicationUser>(new ApplicationDbContext()));
ApplicationUser user = userManager.FindByName(HttpContext.Current.User.Identity.Name);
не возвращает действительный (пользовательский) объект пользователя, проблема заключается в if() заявление.
Ваше обновление выглядит хорошо, и если вы счастливы хранить данные в виде претензий в куки, вы можете пойти с ним, хотя я лично ненавижу try {} поймать блок там.
Вместо этого я делаю вот что:
BaseController.cs
[AuthorizeEx]
public abstract partial class BaseController : Controller
{
public IOwinContext OwinContext
{
get { return HttpContext.GetOwinContext(); }
}
public new ClaimsPrincipal User
{
get { return base.User as ClaimsPrincipal; }
}
public WorkContext WorkContext { get; set; }
}
Я украшаю базовый класс контроллера специальным атрибутом.
AuthorizeExAttribute.cs:
public class AuthorizeExAttribute : AuthorizeAttribute
{
public override void OnAuthorization(AuthorizationContext filterContext)
{
Ensure.Argument.NotNull(filterContext);
base.OnAuthorization(filterContext);
IPrincipal user = filterContext.HttpContext.User;
if (user.Identity.IsAuthenticated)
{
var ctrl = filterContext.Controller as BaseController;
ctrl.WorkContext = new WorkContext(user.Identity.Name);
}
}
}
И WorkContext.cs:
public class WorkContext
{
private string _email;
private Lazy<User> currentUser;
private IAuthenticationService authService;
private ICacheManager cacheManager;
public User CurrentUser
{
get
{
var cachedUser = cacheManager.Get<User>(Constants.CacheUserKeyPrefix + this._email);
if (cachedUser != null)
{
return cachedUser;
}
else
{
var user = currentUser.Value;
cacheManager.Set(Constants.CacheUserKeyPrefix + this._email, user, 30);
return user;
}
}
}
public WorkContext(string email)
{
Ensure.Argument.NotNullOrEmpty(email);
this._email = email;
this.authService = DependencyResolver.Current.GetService<IAuthenticationService>();
this.cacheManager = DependencyResolver.Current.GetService<ICacheManager>();
this.currentUser = new Lazy<User>(() => authService.GetUserByEmail(email));
}
Затем я получаю доступ к WorkContext следующим образом:
public class DashboardController : BaseController
{
public ActionResult Index()
{
ViewBag.User = WorkContext.CurrentUser;
return View();
}
}
Я использую Ninject's Dependency Resolver для решения authService а также cacheManager но вы можете пропустить кэширование и заменить authService на ASP.NET Identity UserManager Я верю.
Я также хотел отдать должное, потому что класс WorkContext в значительной степени вдохновлен проектом NugetGallery.
Бьюсь об заклад, HttpContext.Current.User является нулевым. Итак, вместо этого:
if (HttpContext.Current.User.Identity.IsAuthenticated)
Вы можете попробовать это:
if (HttpContext.Current.Request.IsAuthenticated)
У меня была такая же ошибка.
Моя проблема заключалась в том, что с анонимными пользователями я не устанавливал IIdentity на IPrincipal. Я сделал это только тогда, когда пользователи вошли в систему с именем пользователя. В противном случае IIdentity была нулевой.
Мое решение состояло в том, чтобы всегда устанавливать IIdentity. Если пользователь не аутентифицирован (анонимный пользователь), то для IIdentity.IsAuthenticated устанавливается значение false. В противном случае, правда.
Мой код:
private PrincipalCustom SetPrincipalIPAndBrowser()
{
return new PrincipalCustom
{
IP = RequestHelper.GetIPFromCurrentRequest(HttpContext.Current.Request),
Browser = RequestHelper.GetBrowserFromCurrentRequest(HttpContext.Current.Request),
/* User is not authenticated, but Identity must be set anyway. If not, error occurs */
Identity = new IdentityCustom { IsAuthenticated = false }
};
}