From c728744d9e23a8be7b3a01005354ba2ecb2b742c Mon Sep 17 00:00:00 2001 From: Yuka Higashide Date: Thu, 29 Jan 2026 15:12:56 +0900 Subject: [PATCH] =?UTF-8?q?=E3=82=BB=E3=83=83=E3=82=B7=E3=83=A7=E3=83=B3?= =?UTF-8?q?=E5=88=87=E3=82=8C=E6=99=82=E5=86=8D=E3=83=AD=E3=82=B0=E3=82=A4?= =?UTF-8?q?=E3=83=B3=E5=87=A6=E7=90=86=E3=82=92Trait=E3=81=AB=E7=A7=BB?= =?UTF-8?q?=E5=8B=95?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- app/Http/Controllers/ReceiptController.php | 30 ++++++------ .../Controllers/RegularContractController.php | 11 +++-- app/Http/Middleware/ManagementMiddleware.php | 6 +-- app/Http/Traits/AuthenticatesUser.php | 48 +++++++++++++++++++ 4 files changed, 71 insertions(+), 24 deletions(-) create mode 100644 app/Http/Traits/AuthenticatesUser.php diff --git a/app/Http/Controllers/ReceiptController.php b/app/Http/Controllers/ReceiptController.php index f7778a4..93788aa 100644 --- a/app/Http/Controllers/ReceiptController.php +++ b/app/Http/Controllers/ReceiptController.php @@ -4,6 +4,7 @@ declare(strict_types=1); namespace App\Http\Controllers; +use App\Http\Traits\AuthenticatesUser; use Illuminate\Http\Request; use Illuminate\Http\Response; use Illuminate\Support\Facades\DB; @@ -14,6 +15,8 @@ use function base_path; class ReceiptController extends Controller { + use AuthenticatesUser; + /** * 領収書宛名入力画面を表示する。 * 契約IDに基づき、領収書宛名入力フォームを表示する。 @@ -24,13 +27,12 @@ class ReceiptController extends Controller */ public function input(string $management_code, string $contract_id): mixed { - $user_id = session('user_id'); - if (!$user_id) { - // 未認証アクセス（セッション切れ）を追跡するため、INFOログで記録 - \Log::info("[INFO] " . now()->format('Y-m-d H:i:s') . " 未認証ユーザーによるアクセス（セッション切れ）: 領収書宛名入力画面アクセス"); - return redirect()->guest('/login'); + if ($redirect = $this->handleSessionExpired('領収書宛名入力', false)) { + return $redirect; } + // ヘッダー表示のためユーザー名取得 + $user_id = session('user_id'); $user_name = DB::table('user')->where('user_id', $user_id)->value('user_name'); $management = session('management'); @@ -70,13 +72,12 @@ class ReceiptController extends Controller */ public function issue(Request $request, string $management_code, string $contract_id): mixed { - $user_id = session('user_id'); - if (!$user_id) { - // 未認証アクセス（セッション切れ）を追跡するため、INFOログで記録 - \Log::info("[INFO] " . now()->format('Y-m-d H:i:s') . " 未認証ユーザーによるアクセス（セッション切れ）: 領収書発行"); - return redirect('/login'); + if ($redirect = $this->handleSessionExpired('領収書宛名入力', false)) { + return $redirect; } + $user_id = session('user_id'); + // 既存レコードがある場合、再発行を促す $exists = DB::table('inv_publish')->where('contract_id', $contract_id)->exists(); if ($exists) { @@ -148,12 +149,11 @@ class ReceiptController extends Controller public function download(string $management_code, string $contract_id, bool $is_reissue = true): mixed { try { - $user_id = session('user_id'); - if (!$user_id) { - // 未認証アクセス（セッション切れ）を追跡するため、INFOログで記録 - \Log::info("[INFO] " . now()->format('Y-m-d H:i:s') . " 未認証ユーザーによるアクセス（セッション切れ）: 領収書ダウンロード"); - return redirect('/login'); + if ($redirect = $this->handleSessionExpired('領収書ダウンロード', false)) { + return $redirect; } + + $user_id = session('user_id'); $management = session('management'); $management_id = $management->management_id; diff --git a/app/Http/Controllers/RegularContractController.php b/app/Http/Controllers/RegularContractController.php index 0cfe7ad..ad11744 100644 --- a/app/Http/Controllers/RegularContractController.php +++ b/app/Http/Controllers/RegularContractController.php @@ -4,6 +4,7 @@ declare(strict_types=1); namespace App\Http\Controllers; +use App\Http\Traits\AuthenticatesUser; use Illuminate\Support\Facades\DB; use Illuminate\Http\Request; use Illuminate\Http\Response; @@ -12,6 +13,7 @@ use Carbon\Carbon; class RegularContractController extends Controller { + use AuthenticatesUser; public function showInfo(Request $request) { $user_id = session('user_id'); @@ -89,13 +91,12 @@ class RegularContractController extends Controller * @param Request $request 受信したHTTPリクエスト * @return mixed ビュー応答またはリダイレクト */ - $user_id = session('user_id'); - if (!$user_id) { - // 未認証アクセス（セッション切れ）を追跡するため、INFOログで記録 - \Log::info("[INFO] " . now()->format('Y-m-d H:i:s') . " 未認証ユーザーによるアクセス（セッション切れ）: 定期契約履歴確認画面アクセス"); - return redirect()->guest('/login'); + if ($redirect = $this->handleSessionExpired('定期契約履歴確認')) { + return $redirect; } + // ヘッダー表示のためユーザー名取得 + $user_id = session('user_id'); $user_name = DB::table('user')->where('user_id', $user_id)->value('user_name'); // マルチテナント対応のため、運営元コード、IDを取得 diff --git a/app/Http/Middleware/ManagementMiddleware.php b/app/Http/Middleware/ManagementMiddleware.php index 10ad397..2053210 100644 --- a/app/Http/Middleware/ManagementMiddleware.php +++ b/app/Http/Middleware/ManagementMiddleware.php @@ -6,8 +6,6 @@ use Closure; use Illuminate\Http\Request; use Illuminate\Support\Facades\DB; use Illuminate\Support\Facades\Log; -use Illuminate\Http\Response; -use Illuminate\Http\RedirectResponse; class ManagementMiddleware { @@ -17,10 +15,10 @@ class ManagementMiddleware * * @param Request $request 受信したHTTPリクエスト * @param Closure $next 次のミドルウェア処理 - * @return Response 次の処理のレスポンス + * @return mixed 次の処理のレスポンス * @throws \Symfony\Component\HttpKernel\Exception\HttpException 運営元コードが見つからない場合 */ - public function handle(Request $request, Closure $next): Response|RedirectResponse + public function handle(Request $request, Closure $next): mixed { // マルチテナント対応のため、URLの最初のセグメントを運営元コードとして扱う $path = $request->getPathInfo(); // パス全体を取得 diff --git a/app/Http/Traits/AuthenticatesUser.php b/app/Http/Traits/AuthenticatesUser.php new file mode 100644 index 0000000..5751db5 --- /dev/null +++ b/app/Http/Traits/AuthenticatesUser.php @@ -0,0 +1,48 @@ +management_code ?? request()->segment(1); + + if (!$managementCode) { + \Log::error("[ERROR] " . now()->format('Y-m-d H:i:s') . " 運営元取得失敗 url=" . request()->fullUrl()); + abort(404); + } + + // ログ記録 + $logMessage = "[INFO] " . now()->format('Y-m-d H:i:s') . " 未認証ユーザーアクセス（セッション切れ）"; + if ($screenName) { + $logMessage .= " {$screenName}画面"; + } + $logMessage .= " management_code=" . $managementCode; + \Log::info($logMessage); + + if ($saveIntended) { + // ログイン後に元の画面に戻る + return redirect()->guest(route('swo8_1', ['management_code' => $managementCode])); + } else { + // ログイン後はマイページトップへ + session()->forget('url.intended'); + return redirect()->route('swo8_1', ['management_code' => $managementCode]); + } + } +} -- 2.47.3