ASP.NET C# ページに最後にアクセスした日時をApplicationUser毎に記録したい

Question

###前提・実現したいこと
identity2.0のフォーム認証(個人 ユーザー アカウント)にて、
IdentityUserを継承したApplicationUserに最終アクセス日時「LastAccessDate」のフィールドを追加し、
ページにアクセスする度にLastAccessDateのフィールドをサーバマシンの現在時刻で更新したいと考えております。

###プロジェクト構成

• Visual Studio Professional 2017
• ASP.NET MVC5.2.3
• .NET Framework 4.6.1
• EntityFramework 6.1.3
• ASP.NET Identity 2.2.1
• IIS10

###アドバイスいただきたいこと
現在は、以下のコードにて、ログイン時のみ更新を実現しています。
全てのページにおいてLastAccessDateを更新するための方法がありましたら、
ご教示いただきたいと考えております。

        // POST: /Account/Login
        [HttpPost]
        [AllowAnonymous]
        [ValidateAntiForgeryToken]
        public async Task<ActionResult> Login(LoginViewModel model, string returnUrl)
        {
            if (!ModelState.IsValid)
            {
                return View(model);
            }

            // これは、アカウント ロックアウトの基準となるログイン失敗回数を数えません。
            // パスワード入力失敗回数に基づいてアカウントがロックアウトされるように設定するには、shouldLockout: true に変更してください。
            var result = await SignInManager.PasswordSignInAsync(model.Email, model.Password, model.RememberMe, shouldLockout: false);
            switch (result)
            {
                case SignInStatus.Success:
                    
                    using(var context = new ApplicationDbContext())
                    {
                        context.Users.Single(u => u.Id == User.Identity.GetUserId()).LastAccessDate == DateTime.Now();
                        await context.SaveChangesAsync();
                    }
                    return RedirectToLocal(returnUrl);
                case SignInStatus.LockedOut:
                    return View("Lockout");
                case SignInStatus.RequiresVerification:
                    return RedirectToAction("SendCode", new { ReturnUrl = returnUrl, RememberMe = model.RememberMe });
                case SignInStatus.Failure:
                default:
                    ModelState.AddModelError("", "無効なログイン試行です。");
                    return View(model);
            }
        }

Comments

[退会済みユーザー]

「ページにアクセスする度」ということですと結構負荷が高くて問題ありそうな気がしますがそこは検討済みなのでしょうか？　回答した後やっぱりやめたということになると悲しいのでお聞きしてます。アクションフィルターの利用は検討されたでしょうか？　検討したけどダメだったということであればそのことを書いていただけませんか。

[monagano]

お恥ずかしながら、アクションフィルター機能について存じ上げておりませんでした。アクションフィルターを使用すれば、目的を達せられそうなので、ひとまず着手してみようと思います。アクセス負荷については、200pv/day程度なので、恐らく問題ないと考えております。

Answer 1

「アクションフィルターの利用は検討されたでしょうか？」とか言っておいて、もしできなかったら何なので、自分の環境（Windows 10 64-bit, VS2015 Community Update 3, MVC5, ASP.NET Identity 2, IIS Express 10）で検証してみました。

アクションフィルターで目的は果たせると思います。

ご参考までに、以下の検証に使ったコードをアップしておきます。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using System.Web.Mvc;
using Mvc5App.Models;

namespace Mvc5App.Extensions
{
    public class AccessLogAttribute : ActionFilterAttribute
    {
        public override void OnResultExecuted(ResultExecutedContext filterContext)
        {
            if (filterContext == null)
            {
                throw new ArgumentNullException("OnResultExecuted メソッドの引数が null");
            }

            HttpContextBase context = filterContext.HttpContext;

            if (context.User.Identity.IsAuthenticated)
            {
                using (var db = new ApplicationDbContext())
                {
                    // User.Identity.Name はデフォルトで email と同じになりかつユニーク
                    string userName = context.User.Identity.Name;
                    ApplicationUser user = db.Users.FirstOrDefault(u => u.UserName == userName);
                    // アクセス日時の DB への登録
                }
            }            
        }
    }
}

*** 2017/5/24 17:23 追記 ***

私の 2017/05/24 17:06 のコメントで、「View に渡すならアクションフィルターで OnActionExecuting のタイミングで ViewData にその時点での日時を格納して渡すことができるはずです。検証してから回答欄に書いておきます」と書きました。

すでに解決したとのことですが、コメントに書いた通り当方でも検証してみましたので、検証に使ったコードをご参考までに以下にアップしておきます。

上に書いたコードでは OnResultExecuted のタイミングで処理してますが、その時点で ViewData に追加しても遅すぎで View では取得できないので、OnActionExecuting のタイミングに変えました。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using System.Web.Mvc;
using Mvc5App.Models;

namespace Mvc5App.Extensions
{
    public class AccessLogAttribute : ActionFilterAttribute
    {
        public override void OnActionExecuting(ActionExecutingContext filterContext)
        {
            if (filterContext == null)
            {
                throw new ArgumentNullException("ActionExecutingContext が null");
            }

            HttpContextBase context = filterContext.HttpContext;

            if (context.User.Identity.IsAuthenticated)
            {
                using (var db = new ApplicationDbContext())
                {
                    // User.Identity.Name はデフォルトで email と同じになりかつユニーク
                    string userName = context.User.Identity.Name;
                    ApplicationUser user = db.Users.FirstOrDefault(u => u.UserName == userName);
                    // アクセス日時の DB への登録

                    // ViewData に最新アクセス日時を設定
                    ViewDataDictionary dic = filterContext.Controller.ViewData;
                    dic["LastAccessDateTime"] = DateTime.Now.ToString();
                }
            }

            base.OnActionExecuting(filterContext);
        }
    }
}

View 側では @ViewData["LastAccessDateTime"] で取得できます。

Comments

[monagano]

以下のAction FilterをGlobalFiltersに追加することで、全てのページに目的の処理を追加できました。
ただ、別問題なのですが、DBが更新されても、HttpContext.Userに反映されないため、
View側からUser.Identity.GetLastAccessDate()で最新の値を参照することができずに悩んでおります。

```C#
/// <summary>
/// 自作フィルタ(最終アクセス日時)
/// </summary>
public class AddLastAccessAttribute : ActionFilterAttribute, IDisposable
{
private bool disposed = false;
private ApplicationUserManager _userManager;
public ApplicationUserManager UserManager
{
get
{
return _userManager ?? HttpContext.Current.GetOwinContext().GetUserManager<ApplicationUserManager>();
}
private set
{
_userManager = value;
}
}

~AddLastAccessAttribute()
{
this.Dispose(false);
}

public override void OnResultExecuted(ResultExecutedContext filterContext)
{
if (filterContext.HttpContext.User.Identity.IsAuthenticated)
{
var id = filterContext.HttpContext.User.Identity.GetUserId();
var user = UserManager.FindById(id);
user.LastAccessDate = DateTime.Now;
UserManager.Update(user);

}
}

public void Dispose()
{
this.Dispose(true);
GC.SuppressFinalize(this);
}

protected virtual void Dispose(bool isDisposing)
{
if (!disposed)
{
if (isDisposing)
{
if (_userManager != null)
{
_userManager.Dispose();
_userManager = null;
}
}
disposed = true;
}
}
}
```

[退会済みユーザー]

> View側からUser.Identity.GetLastAccessDate()で最新の値を参照することができずに悩んでおります。

View に渡すならアクションフィルターで OnActionExecuting のタイミングで ViewData にその時点での日時を格納して渡すことができるはずです。検証してから回答欄に書いておきます。

[monagano]

フォローありがとうございます。まさにその方法で実装し、実現しました。
助かりました。