[PR #819] [MERGED] Make sure the user has configured Google Authenticator before enabling 2FA authentication #1070

New issue

Closed

opened 2026-02-25 21:37:08 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 21:37:08 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cypht-org/cypht/pull/819
Author: @Yannick243
Created: 11/3/2023
Status: ✅ Merged
Merged: 2/2/2024
Merged by: @kroky

Base: master ← Head: 2FA-configuration-confirm

📝 Commits (3)

c47a99b Make sure the user has configured Google Authenticator before enabling 2FA authentication
3705352 Make sure the user has configured Google Authenticator before enabling 2FA (Refactor UI on 2 columns)
c787c4a Make sure the user has configured Google Authenticator before enabling 2FA (Unminify CSS)

📊 Changes

4 files changed (+479 additions, -27 deletions)

View changed files

📝 modules/2fa/modules.php (+68 -18)
📝 modules/2fa/setup.php (+17 -5)
📝 modules/2fa/site.css (+219 -4)
➕ modules/2fa/site.js (+175 -0)

📄 Description

This PR ensure that a user has configured Google Authenticator or any Authenticator app before enabling 2FA authentication on next login.

Screen 1:
2factor-0-min

Screen 2:

_{🔄 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/cypht-org/cypht/pull/819 **Author:** [@Yannick243](https://github.com/Yannick243) **Created:** 11/3/2023 **Status:** ✅ Merged **Merged:** 2/2/2024 **Merged by:** [@kroky](https://github.com/kroky) **Base:** `master` ← **Head:** `2FA-configuration-confirm` --- ### 📝 Commits (3) - [`c47a99b`](https://github.com/cypht-org/cypht/commit/c47a99b2bd4282dfb462f2fdec1322837de2d01f) Make sure the user has configured Google Authenticator before enabling 2FA authentication - [`3705352`](https://github.com/cypht-org/cypht/commit/3705352d280031fadbf4c6ffa3bef29bff57ed1b) Make sure the user has configured Google Authenticator before enabling 2FA (Refactor UI on 2 columns) - [`c787c4a`](https://github.com/cypht-org/cypht/commit/c787c4a652003a66e7e979323ec4bc8ba35ca7d6) Make sure the user has configured Google Authenticator before enabling 2FA (Unminify CSS) ### 📊 Changes **4 files changed** (+479 additions, -27 deletions) <details> <summary>View changed files</summary> 📝 `modules/2fa/modules.php` (+68 -18) 📝 `modules/2fa/setup.php` (+17 -5) 📝 `modules/2fa/site.css` (+219 -4) ➕ `modules/2fa/site.js` (+175 -0) </details> ### 📄 Description This PR ensure that a user has configured Google Authenticator or any Authenticator app before enabling 2FA authentication on next login. Related issue: [https://github.com/cypht-org/cypht/issues/772](https://github.com/cypht-org/cypht/issues/772) Screen 1: <img width="880" alt="2factor-0-min" src="https://github.com/cypht-org/cypht/assets/14950700/cd358bda-ab3c-4a64-b189-8c7e1a39a8cd"> Screen 2: ![2factor-1-min](https://github.com/cypht-org/cypht/assets/14950700/629b1de0-0bad-4148-abb3-4d59cfa578bf) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>