[PR #1897] [MERGED] Add back 2fa requirement middleware #1645

Closed
opened 2026-02-05 18:20:07 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/1897
Author: @Boy132
Created: 11/13/2025
Status: Merged
Merged: 11/24/2025
Merged by: @Boy132

Base: mainHead: boy132/fix-2fa-requirement


📝 Commits (2)

  • 4be90a4 add back 2fa requirement middleware
  • 3935d5f Apply suggestions from code review

📊 Changes

3 files changed (+84 additions, -24 deletions)

View changed files

app/Exceptions/Http/TwoFactorAuthRequiredException.php (+18 -0)
app/Http/Middleware/RequireTwoFactorAuthentication.php (+64 -0)
📝 app/Providers/Filament/PanelProvider.php (+2 -24)

📄 Description

Closes #1718


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/pelican-dev/panel/pull/1897 **Author:** [@Boy132](https://github.com/Boy132) **Created:** 11/13/2025 **Status:** ✅ Merged **Merged:** 11/24/2025 **Merged by:** [@Boy132](https://github.com/Boy132) **Base:** `main` ← **Head:** `boy132/fix-2fa-requirement` --- ### 📝 Commits (2) - [`4be90a4`](https://github.com/pelican-dev/panel/commit/4be90a424e85d681cf7bac722c35724302272ff7) add back 2fa requirement middleware - [`3935d5f`](https://github.com/pelican-dev/panel/commit/3935d5fbc5a1887493a0216bb70db9c90c45e1e6) Apply suggestions from code review ### 📊 Changes **3 files changed** (+84 additions, -24 deletions) <details> <summary>View changed files</summary> ➕ `app/Exceptions/Http/TwoFactorAuthRequiredException.php` (+18 -0) ➕ `app/Http/Middleware/RequireTwoFactorAuthentication.php` (+64 -0) 📝 `app/Providers/Filament/PanelProvider.php` (+2 -24) </details> ### 📄 Description Closes #1718 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 18:20:07 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel-pelican-dev#1645