Skip to content

feature/release 0#246

Open
chintakjoshi wants to merge 6 commits into
mainfrom
feature/sms
Open

feature/release 0#246
chintakjoshi wants to merge 6 commits into
mainfrom
feature/sms

Conversation

@chintakjoshi

Copy link
Copy Markdown
Owner

What shipped:

  • app/config.py — three new classes: SmsSettings, MfaRateLimits, MfaSettings; attached as Settings.mfa; three new production guards (mfa.sms.provider != "local", mfa.phone_encryption_key required, mfa.phone_lookup_hash_key required).
  • tests/unit/test_config_mfa_production_guard.py — 28 new tests covering defaults, range validation, provider literal, production guard paths.
  • tests/unit/test_config_production_hardening.py and tests/unit/test_app_production_hardening.py — minimal updates to supply the new required MFA keys when constructing production settings.
  • .env-sample — new MFA__* block with commented-out production-required keys

@chintakjoshi chintakjoshi self-assigned this Apr 24, 2026
@chintakjoshi chintakjoshi added documentation Improvements or additions to documentation enhancement New feature or request labels Apr 24, 2026
@chintakjoshi chintakjoshi linked an issue Apr 24, 2026 that may be closed by this pull request
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Introduce SMS provider for mfa

1 participant