Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: Add multi factor authentication #4345

Open
wants to merge 19 commits into
base: develop
Choose a base branch
from

Conversation

badkeyy
Copy link
Contributor

@badkeyy badkeyy commented Feb 5, 2025

🚨 Contains Database Migration 🚨

Issue

Fixes #4276

Description:

This PR implements multi-factor authentication (MFA) to secure the login process. The new MFA feature adds an extra layer of security by requiring users to provide a second form of authentication, significantly reducing the risk of unauthorized access even if primary credentials (over http) are compromised.

Screenshots

image
image
image
image

@badkeyy badkeyy force-pushed the feature/add-multi-factor-authentication branch from 1b0af23 to d20e151 Compare February 5, 2025 22:36
@badkeyy badkeyy marked this pull request as ready for review February 6, 2025 17:05
@nginxproxymanagerci
Copy link

Docker Image for build 9 is available on
DockerHub
as nginxproxymanager/nginx-proxy-manager-dev:pr-4345

Note: ensure you backup your NPM instance before testing this image! Especially if there are database changes
Note: this is a different docker image namespace than the official image

@jc21 jc21 added the requires-verification Waiting for one or more people to confirm the fix label Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requires-verification Waiting for one or more people to confirm the fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MFA for Nginx Proxy Manager
2 participants