Push notification 2fa #167
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request Summary: Push Notification-Based 2FA
Overview
Implements a configurable push notification-based two-factor authentication (2FA) system using MieWeb Auth App for enhanced security.
Features Added
1. Data Model & Database
Settingmodel for key-value configuration storagekey(primary) andvaluecolumnspush_notification_url→''push_notification_enabled→'false'2. Admin Settings UI (
/settings)3. LDAP Integration (
ldap.conf)AUTH_BACKENDS=sql,notification+NOTIFICATION_URL=${url}/send-notificationAUTH_BACKENDS=sql(default)4. Login Flow with 2FA
After password verification, before session creation:
${push_notification_url}/send-notificationwith:{ "username": "...", "title": "Authentication Request", "body": "Please review and respond...", "actions": [ {"icon": "approve", "title": "Approve", "callback": "approve"}, {"icon": "reject", "title": "Reject", "callback": "reject"} ] }action === 'approve'(case-insensitive) → Login succeedsaction !== 'approve'→ Login denied5. Documentation
Files Changed
New Files (5)
Modified Files (7)
create-a-container/views/partials/header.ejs- Settings navigation linkTesting Performed
Security Considerations