[PR #5469] [CLOSED] Adding OTP Support for Email Login #5468 #6499

Closed
opened 2026-02-05 10:33:55 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/5469
Author: @emirhanuysall
Created: 2/3/2025
Status: Closed

Base: developmentHead: development


📝 Commits (1)

📊 Changes

8 files changed (+215 additions, -2 deletions)

View changed files

app/Access/Controllers/EmailController.php (+120 -0)
📝 app/Access/Controllers/MfaController.php (+15 -0)
📝 app/Access/Mfa/MfaValue.php (+2 -1)
app/Access/Notifications/EmailCodeNotification.php (+32 -0)
resources/views/mfa/email-generate.blade.php (+21 -0)
resources/views/mfa/parts/verify-email.blade.php (+21 -0)
📝 resources/views/mfa/setup.blade.php (+1 -1)
📝 routes/web.php (+3 -0)

📄 Description

Currently, BookStack does not support OTP-based authentication for email logins.

We propose adding an optional OTP feature that integrates with existing MFA systems, allowing users to enter a one-time code sent via email during login.

This would enhance security while maintaining compatibility with BookStack's existing authentication flow.


🔄 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/BookStackApp/BookStack/pull/5469 **Author:** [@emirhanuysall](https://github.com/emirhanuysall) **Created:** 2/3/2025 **Status:** ❌ Closed **Base:** `development` ← **Head:** `development` --- ### 📝 Commits (1) - [`32294eb`](https://github.com/BookStackApp/BookStack/commit/32294eb0d59c91d96af7a20e25f1ff3c83a7e0be) Adding OTP Support for Email Login #5468 ### 📊 Changes **8 files changed** (+215 additions, -2 deletions) <details> <summary>View changed files</summary> ➕ `app/Access/Controllers/EmailController.php` (+120 -0) 📝 `app/Access/Controllers/MfaController.php` (+15 -0) 📝 `app/Access/Mfa/MfaValue.php` (+2 -1) ➕ `app/Access/Notifications/EmailCodeNotification.php` (+32 -0) ➕ `resources/views/mfa/email-generate.blade.php` (+21 -0) ➕ `resources/views/mfa/parts/verify-email.blade.php` (+21 -0) 📝 `resources/views/mfa/setup.blade.php` (+1 -1) 📝 `routes/web.php` (+3 -0) </details> ### 📄 Description Currently, BookStack does not support OTP-based authentication for email logins. We propose adding an optional OTP feature that integrates with existing MFA systems, allowing users to enter a one-time code sent via email during login. This would enhance security while maintaining compatibility with BookStack's existing authentication flow. --- <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 10:33:55 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#6499