Skip to content
This repository was archived by the owner on Feb 9, 2026. It is now read-only.

refactor(auth): 2FA and OTP are disabled in prod environment.#539

Merged
enverern merged 1 commit into
devfrom
refactor/538-deactivate-OTP-and-2FA
Dec 16, 2025
Merged

refactor(auth): 2FA and OTP are disabled in prod environment.#539
enverern merged 1 commit into
devfrom
refactor/538-deactivate-OTP-and-2FA

Conversation

@CotSezer

Copy link
Copy Markdown
Contributor

This PR disables One-Time Password (OTP) validation and Two-Factor Authentication (2FA) enforcement in the production environment. The change ensures that authentication and login flows function without requiring OTP or 2FA while keeping the application logic intact.


Changes Made

  • Disabled real OTP validation logic in production.

    • OTP input is now statically accepted with the value 000000.
  • Disabled 2FA enforcement across authentication flows.

  • Disabled related security-dependent features:

    • Email verification
    • Forgot password reset link functionality
  • Updated authentication flow to allow successful login without OTP/2FA checks.


Acceptance Criteria Coverage

  • ✅ Real OTP validation disabled; fixed OTP value set to 000000
  • ✅ 2FA enforcement disabled (including email verification and password reset links)
  • ✅ Login and authentication flows work correctly without OTP/2FA in production

Testing

  • Verified login flow succeeds without OTP/2FA in production configuration
  • Confirmed OTP input 000000 is accepted
  • Ensured no authentication blockers related to 2FA, email verification, or password reset

Rollback Plan

  • Re-enable OTP validation and 2FA enforcement via configuration and restore original authentication logic.

Closes #538

@CotSezer CotSezer added this to the Milestone 3: Final Milestone milestone Dec 16, 2025
@CotSezer CotSezer self-assigned this Dec 16, 2025
@CotSezer CotSezer added domain: backend Back-end related items type: improvement / refactor an improvement or structural change to existing code priority: urgent Something should be done immediately status:waiting for review [DEPRECATED: Use GitHub Projects status] The task is waiting for to be reviewed. labels Dec 16, 2025

@enverern enverern left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The necessary refactoring has been implemented correctly.

@enverern enverern merged commit 46edb01 into dev Dec 16, 2025
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

domain: backend Back-end related items priority: urgent Something should be done immediately status:waiting for review [DEPRECATED: Use GitHub Projects status] The task is waiting for to be reviewed. type: improvement / refactor an improvement or structural change to existing code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants