29

ASP.Net vNext / MVC6 プロジェクトに取り組んでいます。ASP.Net Identity に慣れてきました。

クラスは明らかに、ApplicationUser追加のユーザー プロパティを追加することになっている場所であり、これは Entity Framework で機能し、追加のプロパティは期待どおりにデータベースに格納されます。

ただし、ビュー内から現在ログインしているユーザーの詳細にアクセスしたい場合に問題が発生します。具体的には_loginPartial.cshtml、メールアドレスが必要なユーザーの Gravatar アイコンを取得して表示する必要があります。

RazorView基本クラスには User プロパティがあり、これはClaimsPrincipal. カスタム プロパティを取得するには、このUserプロパティから に戻るにはどうすればよいですか?ApplicationUser

情報を見つける方法を尋ねているわけではないことに注意してください。ApplicationUser値からを検索する方法を知っていUser.GetUserId()ます。これは、この問題に賢明に取り組む方法についての質問です。具体的には、次のことはしたくありません。

  • ビュー内から任意の種類のデータベース ルックアップを実行します (関心の分離)
  • 現在のユーザーの詳細を取得するには、すべてのコントローラーにロジックを追加する必要があります (DRY 原則)
  • すべての ViewModel に User プロパティを追加する必要があります。

これは、一元化された標準ソリューションが必要な「分野横断的な懸念」のように思えますが、ジグソー パズルのピースが欠けているように感じます。ビュー内からこれらのカスタム ユーザー プロパティを取得する最善の方法は何ですか?

注: MVC チームは、UserName プロパティが常にユーザーの電子メール アドレスに設定されるようにすることで、プロジェクト テンプレート内でこの問題を回避し、ユーザーの電子メール アドレスを取得するためにこの検索を実行する必要がないようにしています! それは私にはちょっとした詐欺のように思えます.私のソリューションでは、ユーザーのログイン名はメールアドレスである場合とそうでない場合があるため、そのトリックに頼ることはできません（後でアクセスする必要がある他のプロパティがあると思います. ）。

4

6 に答える 6

23

元の回答への更新: (これは、op の最初の要件に違反しています。同じ要件がある場合は、元の回答を参照してください) Razor ビューで FullName を参照することで、クレームを変更したり、(私の元のソリューションで) 拡張ファイルを追加したりせずに実行できます。なので：

@UserManager.GetUserAsync(User).Result.FullName

元の回答:

これは、このスタックオーバーフローの質問の短い例であり、このチュートリアルに従っています。

「ApplicationUser.cs」にプロパティが設定されていること、および登録用の該当する ViewModel と View が既にあると仮定します。

追加プロパティとして「FullName」を使用する例:

"AccountController.cs" Register メソッドを次のように変更します。

    public async Task<IActionResult> Register(RegisterViewModel model, string returnUrl = null)
        {
            ViewData["ReturnUrl"] = returnUrl;
            if (ModelState.IsValid)
            {
                var user = new ApplicationUser {
                    UserName = model.Email,
                    Email = model.Email,
                    FullName = model.FullName //<-ADDED PROPERTY HERE!!!
                };
                var result = await _userManager.CreateAsync(user, model.Password);
                if (result.Succeeded)
                {
                    //ADD CLAIM HERE!!!!
                    await _userManager.AddClaimAsync(user, new Claim("FullName", user.FullName)); 

                    await _signInManager.SignInAsync(user, isPersistent: false);
                    _logger.LogInformation(3, "User created a new account with password.");
                    return RedirectToLocal(returnUrl);
                }
                AddErrors(result);
            }

            return View(model);
        }

そして、新しいファイル「Extensions/ClaimsPrincipalExtension.cs」を追加しました

using System.Linq;
using System.Security.Claims;
namespace MyProject.Extensions
    {
        public static class ClaimsPrincipalExtension
        {
            public static string GetFullName(this ClaimsPrincipal principal)
            {
                var fullName = principal.Claims.FirstOrDefault(c => c.Type == "FullName");
                return fullName?.Value;
            }   
        }
    }

次に、プロパティにアクセスする必要があるビューに次を追加します。

@using MyProject.Extensions

必要に応じて次のように呼び出します。

@User.GetFullName()

これに関する 1 つの問題は、データベースに FullName プロパティが含まれていても、「FullName」を表示するために、現在のテスト ユーザーを削除してから再登録する必要があったことです。

于 2016-07-09T18:27:55.770 に答える
18

そのためには、 User の Claims プロパティを使用する必要があると思います。私は良い記事を見つけました: http://benfoster.io/blog/customising-claims-transformation-in-aspnet-core-identity

ユーザークラス

public class ApplicationUser : IdentityUser
{
    public string MyProperty { get; set; }
}

MyProperty を Authenticated User の Claims に入れましょう。この目的のために、UserClaimsPrincipalFactory をオーバーライドしています

public class MyUserClaimsPrincipalFactory : UserClaimsPrincipalFactory<ApplicationUser, IdentityRole>
{
    public MyUserClaimsPrincipalFactory (
        UserManager<ApplicationUser> userManager,
        RoleManager<IdentityRole> roleManager,
        IOptions<IdentityOptions> optionsAccessor) : base(userManager, roleManager, optionsAccessor)
    {
    }

    public async override Task<ClaimsPrincipal> CreateAsync(ApplicationUser user)
    {
        var principal = await base.CreateAsync(user);

        //Putting our Property to Claims
        //I'm using ClaimType.Email, but you may use any other or your own
        ((ClaimsIdentity)principal.Identity).AddClaims(new[] {
        new Claim(ClaimTypes.Email, user.MyProperty)
    });

        return principal;
    }
}

Startup.cs に UserClaimsPrincipalFactory を登録する

public void ConfigureServices(IServiceCollection services)
{
    //...
    services.AddScoped<IUserClaimsPrincipalFactory<ApplicationUser>, MyUserClaimsPrincipalFactory>();
    //...
}

これで、このようにプロパティにアクセスできます

User.Claims.FirstOrDefault(v => v.Type == ClaimTypes.Email).Value;

拡張機能を作成する場合があります

namespace MyProject.MyExtensions
{
    public static class MyUserPrincipalExtension
    {
        public static string MyProperty(this ClaimsPrincipal user)
        {
            if (user.Identity.IsAuthenticated)
            {
                return user.Claims.FirstOrDefault(v => v.Type == ClaimTypes.Email).Value;
            }

            return "";
        }
    }
}

View に @Using を追加する必要があります (グローバル _ViewImport.cshtml に追加します)。

@using MyProject.MyExtensions

最後に、このプロパティを任意のビューでメソッド呼び出しとして使用できます

@User.MyProperty()

この場合、ユーザー情報を取得するためのデータベースへの追加のクエリはありません。

于 2016-04-04T04:44:29.037 に答える
2

同じ問題と懸念がありますが、ClaimsPrincipal の拡張メソッドを作成する代わりに別のソリューションを選択し、拡張メソッドでカスタム ユーザー プロパティを取得できるようにしました。

ここに私の拡張方法があります:

public static class PrincipalExtensions
{
    public static string ProfilePictureUrl(this ClaimsPrincipal user, UserManager<ApplicationUser> userManager)
    {
        if (user.Identity.IsAuthenticated)
        {
            var appUser = userManager.FindByIdAsync(user.GetUserId()).Result;

            return appUser.ProfilePictureUrl;
        }

        return "";
    }
}

次に私のビュー (LoginPartial ビューも) で、UserManager を注入し、その UserManager を拡張メソッドに転送します。

@inject Microsoft.AspNet.Identity.UserManager<ApplicationUser> userManager;
<img src="@User.ProfilePictureUrl(userManager)">

このソリューションは、懸念の分離、DRY、ViewModel への変更なしという 3 つの要件も満たしていると思います。ただし、このソリューションはシンプルで、ViewComponents だけでなく標準ビューでも使用できますが、私はまだ満足していません。今私の見解では、 @User.ProfilePictureUrl(userManager)と書くことができますが、 @User.ProfilePictureUrl()だけを書くことができるように頼むことはあまりないと思います 。

関数を挿入せずに UserManager (または IServiceProvider) を拡張メソッドで使用できるようにすることができれば、問題は解決しますが、それを行う方法はわかりません。

于 2016-03-31T09:53:19.843 に答える
2

OK、これが私が最終的にやった方法です。View Componentsと呼ばれる MVC6 の新機能を使用しました。これらは部分ビューに少し似ていますが、「ミニ コントローラー」が関連付けられています。ビュー コンポーネントは、モデル バインディングに参加しない軽量のコントローラーですが、依存性注入を使用して、コンストラクター パラメーターで何かを渡すことができます。その後、ビュー モデルを構築し、それを部分ビューに渡すことができます。したがって、たとえば、UserManagerビュー コンポーネントにインスタンスを挿入し、それを使用しApplicationUserて現在のユーザーのオブジェクトを取得し、それを部分ビューに渡すことができます。

コードでは次のようになります。まず、/ViewComponentsディレクトリに存在するビュー コンポーネント:

public class UserProfileViewComponent : ViewComponent
    {
    readonly UserManager<ApplicationUser> userManager;

    public UserProfileViewComponent(UserManager<ApplicationUser> userManager)
        {
        Contract.Requires(userManager != null);
        this.userManager = userManager;
        }

    public IViewComponentResult Invoke([CanBeNull] ClaimsPrincipal user)
        {
        return InvokeAsync(user).WaitForResult();
        }

    public async Task<IViewComponentResult> InvokeAsync([CanBeNull] ClaimsPrincipal user)
        {
        if (user == null || !user.IsSignedIn())
            return View(anonymousUser);
        var userId = user.GetUserId();
        if (string.IsNullOrWhiteSpace(userId))
            return View(anonymousUser);
        try
            {
            var appUser = await userManager.FindByIdAsync(userId);
            return View(appUser ?? anonymousUser);
            }
        catch (Exception) {
        return View(anonymousUser);
        }
        }

    static readonly ApplicationUser anonymousUser = new ApplicationUser
        {
        Email = string.Empty,
        Id = "anonymous",
        PhoneNumber = "n/a"
        };
    }

userManagerコンストラクター パラメーターは MVC フレームワークによって挿入されることに注意してください。Startup.csこれは新しいプロジェクトでデフォルトで構成されているため、構成を行う必要はありません。

Invokeビュー コンポーネントは、当然のことながら、メソッドまたはその非同期バージョンを呼び出すことによって呼び出されます。このメソッドは、可能な場合は を取得しますApplicationUser。それ以外の場合は、安全なデフォルトが事前設定された匿名ユーザーを使用します。このユーザーをビュー モデルの partiel ビューに使用します。ビューは存在し、次の/Views/Shared/Components/UserProfile/Default.cshtmlように始まります。

@model ApplicationUser

<div class="dropdown profile-element">
    <span>
        @Html.GravatarImage(Model.Email, size:80)
    </span>
    <a data-toggle="dropdown" class="dropdown-toggle" href="#">
        <span class="clear">
            <span class="block m-t-xs">
                <strong class="font-bold">@Model.UserName</strong>
            </span> <span class="text-muted text-xs block">@Model.PhoneNumber <b class="caret"></b></span>
        </span>
    </a>

</div>

そして最後に、次の_Navigation.cshtmlように部分ビュー内からこれを呼び出します。

@await Component.InvokeAsync("UserProfile", User)

これは、次の理由により、元の要件をすべて満たしています。

  1. ビューではなく、コントローラー (ビュー コンポーネントはコントローラーの一種) でデータベース ルックアップを実行しています。さらに、フレームワークがリクエストをすでに認証しているため、データはすでにメモリ内にある可能性があります。別のデータベース ラウンド トリップが実際に発生するかどうかは調べていません。おそらく気にすることはありませんが、知っている人がいる場合はお知らせください。
  2. ロジックは明確に定義された 1 つの場所にあります。DRY原則が尊重されます。
  3. 他のビュー モデルを変更する必要はありません。

結果！誰かがこれが役に立つことを願っています...

于 2016-03-05T13:38:44.400 に答える
1

それについて尋ねられたので、別の（MVC5/EF6）プロジェクトではありますが、最終的な解決策を投稿しています。

まず、インターフェイスを定義しました。 

public interface ICurrentUser
    {
    /// <summary>
    ///     Gets the display name of the user.
    /// </summary>
    /// <value>The display name.</value>
    string DisplayName { get; }

    /// <summary>
    ///     Gets the login name of the user. This is typically what the user would enter in the login screen, but may be
    ///     something different.
    /// </summary>
    /// <value>The name of the login.</value>
    string LoginName { get; }

    /// <summary>
    ///     Gets the unique identifier of the user. Typically this is used as the Row ID in whatever store is used to persist
    ///     the user's details.
    /// </summary>
    /// <value>The unique identifier.</value>
    string UniqueId { get; }

    /// <summary>
    ///     Gets a value indicating whether the user has been authenticated.
    /// </summary>
    /// <value><c>true</c> if this instance is authenticated; otherwise, <c>false</c>.</value>
    bool IsAuthenticated { get; }

次に、それを具象クラスに実装します。

/// <summary>
///     Encapsulates the concept of a 'current user' based on ASP.Net Identity.
/// </summary>
/// <seealso cref="MS.Gamification.DataAccess.ICurrentUser" />
public class AspNetIdentityCurrentUser : ICurrentUser
    {
    private readonly IIdentity identity;
    private readonly UserManager<ApplicationUser, string> manager;
    private ApplicationUser user;

    /// <summary>
    ///     Initializes a new instance of the <see cref="AspNetIdentityCurrentUser" /> class.
    /// </summary>
    /// <param name="manager">The ASP.Net Identity User Manager.</param>
    /// <param name="identity">The identity as reported by the HTTP Context.</param>
    public AspNetIdentityCurrentUser(ApplicationUserManager manager, IIdentity identity)
        {
        this.manager = manager;
        this.identity = identity;
        }

    /// <summary>
    ///     Gets the display name of the user. This implementation returns the login name.
    /// </summary>
    /// <value>The display name.</value>
    public string DisplayName => identity.Name;

    /// <summary>
    ///     Gets the login name of the user.
    ///     something different.
    /// </summary>
    /// <value>The name of the login.</value>
    public string LoginName => identity.Name;

    /// <summary>
    ///     Gets the unique identifier of the user, which can be used to look the user up in a database.
    ///     the user's details.
    /// </summary>
    /// <value>The unique identifier.</value>
    public string UniqueId
        {
        get
            {
            if (user == null)
                user = GetApplicationUser();
            return user.Id;
            }
        }

    /// <summary>
    ///     Gets a value indicating whether the user has been authenticated.
    /// </summary>
    /// <value><c>true</c> if the user is authenticated; otherwise, <c>false</c>.</value>
    public bool IsAuthenticated => identity.IsAuthenticated;

    private ApplicationUser GetApplicationUser()
        {
        return manager.FindByName(LoginName);
        }
    }

最後に、DI カーネルで次の構成を行います (Ninject を使用しています)。

        kernel.Bind<ApplicationUserManager>().ToSelf()
            .InRequestScope();
        kernel.Bind<ApplicationSignInManager>().ToSelf().InRequestScope();
        kernel.Bind<IAuthenticationManager>()
            .ToMethod(m => HttpContext.Current.GetOwinContext().Authentication)
            .InRequestScope();
        kernel.Bind<IIdentity>().ToMethod(p => HttpContext.Current.User.Identity).InRequestScope();
        kernel.Bind<ICurrentUser>().To<AspNetIdentityCurrentUser>();

次に、現在のユーザーにアクセスしたいときはいつでも、 type のコンストラクター パラメーターを追加してコントローラーに挿入しますICurrentUser。

このソリューションは、問題を適切にカプセル化し、コントローラーが EF に直接依存することを回避できるため、気に入っています。

于 2016-08-30T17:39:05.380 に答える
0

現在のユーザーの名前で (たとえば Entity Framework を使用して) 検索を行う必要があります。

HttpContext.Current.User.Identity.Name
于 2016-03-03T20:04:37.150 に答える