webdevqa.jp.net

MVC5およびOWINを使用したカスタムID

MVC5およびOWIN認証を使用して、WebサイトのApplicationUserにカスタムプロパティを追加しようとしています。 https://stackoverflow.com/a/10524305/264607 を読みましたが、新しいプロパティに簡単にアクセスできるようにベースコントローラーと統合する方法が気に入っています。私の問題は、HTTPContext.Current.Userプロパティを新しいIPrincipalに設定すると、null参照エラーが発生することです。

[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://blogs.msdn.com/b/webdev/archive/2013/10/16/customizing-profile-information-in-asp-net-identity-in-vs-2013-templates.aspx

http://blogs.msdn.com/b/webdev/archive/2013/07/03/understanding-owin-forms-authentication-in-mvc-5.aspx

2番目のリンクのコメントは、おそらくクレーム( http://msdn.Microsoft.com/en-us/library/ms734687.aspx?cs-save-lang=1&cs-lang=csharp =)が、リンク先の記事ではそれらをIPrincipal(これはHttpContext.Current.User is)、またはパイプラインのClaimsIdentityUserの具象クラス)に追加することになっている場所。クレームの使用に傾倒していますが、これらの新しいクレームをユーザーに追加する場所を知る必要があります。

クレームが進むべき方法であっても、必要なすべてを実装しているように見えるため、カスタムIPrincipalで間違っていることについて興味があります。

22
BlackICE

私はClaimsベースのセキュリティを使用して何かを動作させることができますので、あなたが何かをすぐに終わらせることを探しているなら、私は現時点で持っているものです:

AccountController(私のものはSignInAsyncメソッド内にあります)のログインプロセスで、UserManagerによって作成されたIDに新しいクレームを追加します。

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#code-snippet-1


IPrincipalの問題の更新

Identityプロパティまで追跡しました。問題は、Identityプロパティを設定していないPatientPortalPrincipalクラスのデフォルトコンストラクターを提供していたことです。私がやったことは、デフォルトのコンストラクタを削除し、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;
    }
}

これですべてが機能します!

18
BlackICE

チェックポイントで_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()ステートメントにあります。

更新は問題なく行われ、Cookieにデータをクレームとして保存することができれば、それを使用できますが、_try {}_ catchブロックは個人的には嫌いです。

私が代わりにやることはこれです:

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の依存関係リゾルバーを使用してauthServicecacheManagerを解決していますが、キャッシュをスキップしてauthServiceをASP.NET Identity UserManagerと置き換えることができます。

また、WorkContextクラスはNugetGalleryプロジェクトから大きな影響を受けているため、当然のこととしてクレジットを付けたかったのです。

5
LukeP

私はHttpContext.Current.Userがnullであると確信しています。したがって、これの代わりに:

if (HttpContext.Current.User.Identity.IsAuthenticated)

これを試すことができます:

if (HttpContext.Current.Request.IsAuthenticated)
3
Brock Allen

同じエラーが発生しました。

私の問題は、匿名ユーザーではIPrincipalにIIdentityを設定していないことでした。ユーザーがユーザー名でログインしたときにのみこれを行いました。それ以外の場合、IIdentityはヌルでした。

私の解決策は、IIdentityを常に設定することでした。ユーザーが認証されていない場合(匿名ユーザー)、IIdentity.IsAuthenticatedはfalseに設定されます。そうでなければ、true。

私のコード:

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 }
     };
}
0
FrenkyB