admin認証済みの場合はメンテナンスモードを回避するよう変更

[izayoi256]

概要(Overview・Refs Issue)

#4839

(マージ先は #4901 の方針に従って4.0にしました)

方針(Policy)

セッションから管理画面の認証情報を取得する処理は以下を流用。
https://github.com/symfony/security/blob/v3.4.42/Http/Firewall/ContextListener.php

テスト（Test)

ユニットテストなし。

相談（Discussion）

従来は\Symfony\Component\HttpKernel\Kernel::boot()の前に処理していたのが、その後の処理になるが問題ないか。

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[kiy0taka]

@izayoi256 #4057 と同じような実装だと思いますので、この仕様だとキャッシュ生成の制御ができないため、申し訳ないですが取り込むことができないです。

[izayoi256]

あぁ、 #4057 があったんですね気付きませんでした。やはり現状の実装はキャッシュを考慮してますよね。
そうなると #4839 はwontfixでクローズでもいいんじゃないでしょうか。

[yKazihara]

プルリクいただきありがとうございました。
社内で検討の結果、現在のメンテナンスモードとは別に管理者のみ閲覧できるモードの機能追加のIssueを立てさせていただきます。
#4839 はwontfixでクローズさせていただきます。

[kiy0taka]

@izayoi256 別の方法を考えてみたので、ご意見いただければと思います。#4961