[PR #447] [MERGED] [Feature] Two Factor Authorization #8092

New issue

Closed

opened 2026-03-12 23:36:48 +03:00 by kerem · 0 comments

kerem commented

2026-03-12 23:36:48 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/0xJacky/nginx-ui/pull/447
Author: @0xJacky
Created: 7/23/2024
Status: ✅ Merged
Merged: 7/23/2024
Merged by: @0xJacky

Base: dev ← Head: feat/2fa

📝 Commits (8)

5abd9b7 feat: login 2fa
802d05f chore: update translations
3a22861 feat: 2FA authorization for web terminal
a5dda59 chore: update translations
b68e8a1 enhance: do not require 2fa in 2min
afd6bf4 fix: do not popup otp modal if user not enable 2fa
46e78f4 chore: update docs
bcff00c feat: disable enroll 2fa in demo mode

📊 Changes

57 files changed (+2769 additions, -446 deletions)

View changed files

📝 .idea/vcs.xml (+0 -1)
📝 api/api.go (+5 -0)
📝 api/user/auth.go (+31 -10)
📝 api/user/casdoor.go (+1 -1)
➕ api/user/otp.go (+208 -0)
📝 api/user/router.go (+18 -10)
📝 app.example.ini (+8 -0)
📝 app/components.d.ts (+5 -0)
📝 app/package.json (+3 -0)
📝 app/pnpm-lock.yaml (+93 -0)
📝 app/src/api/auth.ts (+4 -3)
➕ app/src/api/otp.ts (+29 -0)
➕ app/src/components/OTP/OTPAuthorization.vue (+78 -0)
➕ app/src/components/OTP/useOTPModal.ts (+92 -0)
➕ app/src/components/OTPInput/OTPInput.vue (+72 -0)
📝 app/src/language/LINGUAS (+1 -1)
📝 app/src/language/en/app.po (+139 -34)
📝 app/src/language/es/app.po (+140 -34)
📝 app/src/language/fr_FR/app.po (+140 -34)
📝 app/src/language/ko_KR/app.po (+140 -34)

...and 37 more files

📄 Description

Add 2FAS (TOTP) #344, #419 .
2FA authorization for Web Terminal #394

_{🔄 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/0xJacky/nginx-ui/pull/447 **Author:** [@0xJacky](https://github.com/0xJacky) **Created:** 7/23/2024 **Status:** ✅ Merged **Merged:** 7/23/2024 **Merged by:** [@0xJacky](https://github.com/0xJacky) **Base:** `dev` ← **Head:** `feat/2fa` --- ### 📝 Commits (8) - [`5abd9b7`](https://github.com/0xJacky/nginx-ui/commit/5abd9b75bbae19df48566581ef51cade9c80993d) feat: login 2fa - [`802d05f`](https://github.com/0xJacky/nginx-ui/commit/802d05f692b18ef883b86f631ea0f55ffc2407a5) chore: update translations - [`3a22861`](https://github.com/0xJacky/nginx-ui/commit/3a22861640f4e19ad6efa60d02fb813a0bd00c5c) feat: 2FA authorization for web terminal - [`a5dda59`](https://github.com/0xJacky/nginx-ui/commit/a5dda59ad513b641dd2aa36ea2ffd078bffc15d3) chore: update translations - [`b68e8a1`](https://github.com/0xJacky/nginx-ui/commit/b68e8a140374f79c47745bb6514d0274184a661e) enhance: do not require 2fa in 2min - [`afd6bf4`](https://github.com/0xJacky/nginx-ui/commit/afd6bf4268da9ce8be446a0014960a8766168ed1) fix: do not popup otp modal if user not enable 2fa - [`46e78f4`](https://github.com/0xJacky/nginx-ui/commit/46e78f440555bc86b630916da027720582fe098e) chore: update docs - [`bcff00c`](https://github.com/0xJacky/nginx-ui/commit/bcff00c47bdd38a4fa2e4d3004af7ba55aef59bf) feat: disable enroll 2fa in demo mode ### 📊 Changes **57 files changed** (+2769 additions, -446 deletions) <details> <summary>View changed files</summary> 📝 `.idea/vcs.xml` (+0 -1) 📝 `api/api.go` (+5 -0) 📝 `api/user/auth.go` (+31 -10) 📝 `api/user/casdoor.go` (+1 -1) ➕ `api/user/otp.go` (+208 -0) 📝 `api/user/router.go` (+18 -10) 📝 `app.example.ini` (+8 -0) 📝 `app/components.d.ts` (+5 -0) 📝 `app/package.json` (+3 -0) 📝 `app/pnpm-lock.yaml` (+93 -0) 📝 `app/src/api/auth.ts` (+4 -3) ➕ `app/src/api/otp.ts` (+29 -0) ➕ `app/src/components/OTP/OTPAuthorization.vue` (+78 -0) ➕ `app/src/components/OTP/useOTPModal.ts` (+92 -0) ➕ `app/src/components/OTPInput/OTPInput.vue` (+72 -0) 📝 `app/src/language/LINGUAS` (+1 -1) 📝 `app/src/language/en/app.po` (+139 -34) 📝 `app/src/language/es/app.po` (+140 -34) 📝 `app/src/language/fr_FR/app.po` (+140 -34) 📝 `app/src/language/ko_KR/app.po` (+140 -34) _...and 37 more files_ </details> ### 📄 Description - [x] Add 2FAS (TOTP) #344, #419 . - [x] 2FA authorization for Web Terminal #394 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>