Controllerにログインページに直接飛ぶアクションを追加すると正常な挙動をしない。

Question

前提・実現したいこと

https://teratail.com/questions/347736
この質問を前提としています。
この質問の回答により、「ログインページを初期画面にするにはControllerそのものに[Authorize]属性をつければ匿名ユーザーがそのそのアクションをしてきた際にログインページにリダイレクトすることができるとわかりました。

ですがControllerに直接ログインページに飛ぶアクションを追加しそれをStartUpから動くように設定すると正常に動いてくれない理由がわかりません。　
後学のために[Authorize]属性を使わず匿名ユーザーがアクセスしてきた場合ログインページに飛ばす方法を知りたいです。

##試したこと
HomeControllerにログインページのcshtmlに飛ぶアクションを追加しました。
次にStartup.csをこれが最初に実行されるアクションになるよう変更しました。

#####HomeContolloer

using Microsoft.AspNetCore.Mvc;
using Microsoft.Extensions.Logging;
using System;
using System.Collections.Generic;
using System.Diagnostics;
using System.Linq;
using System.Threading.Tasks;　
using プロジェクト.Models;
using Microsoft.AspNetCore.Authorization;

namespace プロジェクト.Controllers
{

    [Authorize]
    public class HomeController : Controller
    {
        private readonly ILogger<HomeController> _logger;


        public HomeController(ILogger<HomeController> logger)
        {
            _logger = logger;
        }


        
        public IActionResult Index()
        {
            return View();
        }      

        public IActionResult Privacy()
        {
            return View();
        }


　　　　//ここから追加
        public IActionResult Login()
        {
            return View("Areas/Identity/Pages/Account/Login.cshtml");
        }
　　　  //ここまで追加


        [ResponseCache(Duration = 0, Location = ResponseCacheLocation.None, NoStore = true)]
        public IActionResult Error()
        {
            return View(new ErrorViewModel { RequestId = Activity.Current?.Id ?? HttpContext.TraceIdentifier });
        }
    }
}

#####Startup.cs

using Microsoft.AspNetCore.Builder;
using Microsoft.AspNetCore.Hosting;
using Microsoft.AspNetCore.HttpsPolicy;
using Microsoft.AspNetCore.Identity;
using Microsoft.AspNetCore.Identity.UI;
using Microsoft.EntityFrameworkCore;
using Microsoft.Extensions.Configuration;
using Microsoft.Extensions.DependencyInjection;
using Microsoft.Extensions.Hosting;
using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;
using プロジェクト.Data;

namespace プロジェクト
{
    public class Startup
    {
        public Startup(IConfiguration configuration)
        {
            Configuration = configuration;
        }

        public IConfiguration Configuration { get; }

        // This method gets called by the runtime. Use this method to add services to the container.
        public void ConfigureServices(IServiceCollection services)
        {
            services.AddRazorPages();

            services.AddDbContext<ApplicationDbContext>(options =>
                options.UseSqlServer(                                                                                                           
                    Configuration.GetConnectionString("DefaultConnection")));
            services.AddDatabaseDeveloperPageExceptionFilter();

            services.AddDefaultIdentity<IdentityUser>(options => options.SignIn.RequireConfirmedAccount = true)
                .AddEntityFrameworkStores<ApplicationDbContext>();
            services.AddControllersWithViews();
        }

        // This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
        public void Configure(IApplicationBuilder app, IWebHostEnvironment env)
        {
            if (env.IsDevelopment())
            {
                app.UseDeveloperExceptionPage();
                app.UseMigrationsEndPoint();
            }
            else
            {
                app.UseExceptionHandler("/Home/Error");
                // The default HSTS value is 30 days. You may want to change this for production scenarios, see https://aka.ms/aspnetcore-hsts.
                app.UseHsts();
            }
            app.UseHttpsRedirection();
            app.UseStaticFiles();

            app.UseRouting();

            app.UseAuthentication();
            app.UseAuthorization();

            app.UseEndpoints(endpoints =>
            {
                endpoints.MapRazorPages();
                endpoints.MapControllerRoute(
                    name: "default",
                    pattern: "{controller=Home}/{action=Login}/{id?}");
            });
        }
    }
}

以下ログインページのコード
#####Login.cshtml

@page

@model LoginModel

@{
    ViewData["Message3"] = "ログイン";
}

<h1> @ViewData["Message3"] </h1>　
<div class="row">
    <div class="col-md-4">
        <section>
            <form id="account" method="post">
                <h4>IDを入力してログインします</h4>
                <hr />
                <div class="form-group">
                    <label asp-for="Input.Email">ID</label>
                    <input asp-for="Input.Email" class="form-control" />
                    <span 9="Input.Email" class="text-danger"></span>
                </div>
                <div class="form-group">
                    <label asp-for="Input.Password">パスワード</label>
                    <input asp-for="Input.Password" class="form-control" />
                    <span asp-validation-for="Input.Password" class="text-danger"></span>
                </div>
                <div class="form-group">
                    <div class="checkbox">
                        <label asp-for="Input.RememberMe">

                            <input asp-for="Input.RememberMe" />
                             @Html.DisplayNameFor(m => m.Input.RememberMe) 
                           
                        </label>
                    </div>
                </div>
                <div class="form-group">
                    <button type="submit" class="btn btn-primary">Log in</button>
                </div>
        
            </form>
        </section>
    </div>
</div>

@section Scripts {
    <partial name="_ValidationScriptsPartial" />
}

起きてるエラー

まずこの状態で起動するとLogin.cshtmlの　6行目　ViewData["Message3"] = "ログイン";　の部分に　NullReferenceException: Object reference not set to an instance of an object　が表示され実行できません。

次に@pageディレクティブを外し9行目にタイトルを直打ちした状態で起動するとログインページ自体に飛ぶことはできましたがログインボタンを押した後画面が遷移しません。
ブレイクポイントを置いて確認したところLogin.cshtml.csが動いていないことがわかりました。

開発環境

Windows 10 Pro 19043.1083
Microsoft Visual Studio Community 2019 Version 16.9.4
ログインに関するコードはASP.netIdentityの「個別のアカウント」を使い生成しました。

Answer 1

ブレイクポイントを置いて確認したところLogin.cshtml.csが動いていないことがわかりました。

それが NullReferenceException の原因でしょう。

View(string) というオーバーロードは "Creates a ViewResult object by specifying a viewName." ということで、ページモデル (cshtml.cs) は動かないのですが、その場合 ViewData.get は null を返すよいうことのようです。

Controller Class
https://docs.microsoft.com/en-us/dotnet/api/microsoft.aspnetcore.mvc.controller?view=aspnetcore-5.0

Redirect(string) と使って Login ページにリダイレクトすれば動くと思います。string は Login ページの url にしてください。Razor ページなので Areas/Identity/Pages/Account/Login ではないことに注意。以下のような感じ。

public IActionResult Index()
{
    if (!User.Identity.IsAuthenticated)
    {
        return Redirect("/Identity/Account/Login");
    }
    return View();
}

上記で、ユーザーが Home/Index にアクセスしてくると、そのユーザーが認証されない場合 (User.Identity.IsAuthenticated が false) は Login ページにリダイレクトされるはずです。

Comments

[Mk-ky]

ありがとうございます！
完全に思い通りの動きにできました！
二つ疑問があり一つはcshtmlの中ででViewDataに値を格納しているのにnullになるのはなぜですか？
二つ目はView@modelディレクティブでLoginmodelを指定しているのにcshtml.csが動かないのはなぜですか？
viewだけが返されてもviewの中のコードにそれらが書かれているはずなので動くのではないのかと思いました。

[退会済みユーザー]

普通ではやらないことをして普通でないエラーが出ているよいうな気がしますが・・・

> cshtmlの中ででViewDataに値を格納しているのにnullになるのはなぜですか？

Razor の .cshtml (ページファイル) と .cshtml.cs (ページモデル) は一体モノで、MVC の .cshtml (ビュー) とは違うものだから、MVC のビューのように retrun View("xxx.cshtml") とはできないということだと思います。

@page (Razor ディレクティブ) を外して動きが変わるということがそれを裏付けているのでは？　外すことによって ASP.NET がそのページを Razor ページとして処理しなくなる（なので、ViewData["xxx"] で NullReferenceException が出なくなる）のではないかと思います。

以上は想像です。その想像を裏付ける公式ドキュメントは見つけられていません。（普通そんなことはしないので、そんなことが書いてあるドキュメントは見つからないのではないかと勝手に思ってます）

> View@modelディレクティブでLoginmodelを指定しているのにcshtml.csが動かないのはなぜですか？

LoginModel は渡せてません。ViewData 周りのコードをコメントアウトして NullReferenceException を回避して試してみてください。asp-for="Input.Email" あたりでエラーになると思います。

そのあたりは MVC のビューでも同じで、アクションメソッド無しでは Model を使うあたりでエラーになるはずです。

とにかく普通はやらないことをやって、それで出てくるエラーが何故かを詮索してもあまり実のある話にはならないかと思います。

このあたりで本スレッドの話は終わりにしませんか。

[Mk-ky]

ありがとうございます。
なにか勉強になるかと思ったのですが、こちらも変なことを試してみて起きる変なエラーについて質問して時間を取らせてしまい申し訳ありませんでした。