Steven
b5170ad1ce
CI / backend (push) Canceled after 0s
短信登录关掉后，生产没有任何注册入口——这一版补上最普通的那条路：
填邮箱、收验证码、设密码，注册完直接进首页。
先发码再注册，不是"注册时顺带发信"
POST /auth/email/code 只做三件事：查重、限流、发码。
填完密码才被告知"这个邮箱注册过了"最气人，所以占用（2019）在发码前就回。
邮箱既是账号名也是唯一联系方式，所以必须真验一次码
验证通过时间落 app_user.email_verified_at，/auth/me 回 email_verified。
"随手填一个"会让账号变成没人能找回的孤岛。
app_user.phone 改为可空（迁移 0011）
邮箱注册的账号本来就没有手机号。user.phone 对这类账号回空串，
成员列表同理，不再伪装成一串星号。
防刷
同邮箱 60 秒一次（2022）、验证码 10 分钟有效、错 5 次即作废（2021）。
6 位数可以被刮，不设上限等于把注册口子交给脚本。
本机联调开关 EMAIL_CODE_DEV_ECHO
为 true 时把码回给调用方且不发真信——否则每跑一次 e2e 就往真实收件箱塞一封。
生产打开会让 validate_runtime() 拒绝启动。
前端
新增 /register（邮箱、验证码、密码、确认、昵称；60 秒倒计时；
服务端关着注册时整页只说"联系管理员开通"），登录页按 /auth/methods
决定要不要显示「还没有账号？用邮箱注册」。
验证
pytest 139 passed、ruff check / format 全绿
tsc --noEmit 0、vite build 通过
e2e register.mjs 7 项：登录页入口 → 拿码 → 注册落进首页 → 退出用邮箱+密码登回来
→ 同一邮箱再发码回 2019 → 控制台零报错
e2e login-email-smtp.mjs 15 项回归通过（含真发一封测试信）
e2e account.mjs 14 项、knowledge.mjs 全绿
顺带修的两处测试
test_account 的字段白名单补 email_verified（接口确实多了这个字段）；
test_config_prod 用 Settings(_env_file=None, ...) 构造，别被本机 .env 里的
EMAIL_CODE_DEV_ECHO=true 带偏——那个开关在生产是硬错误，本机却是跑 e2e 必需的。
48 lines
1.8 KiB
Python
48 lines
1.8 KiB
Python
"""生产配置校验：固定验证码必须显式开关，否则启动即失败。
|
|
|
|
这两条是「部署纪律」的守门员：生产一旦误用 dev 验证码，等于任意手机号可登录。
|
|
现在的规则更简单——**短信验证码登录默认关着**，关着时 SMS_PROVIDER 是 dev 也无所谓，
|
|
因为那条路根本不通；只有在 SMS_LOGIN_ENABLED=true 时才要求真通道。
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
import pytest
|
|
|
|
from app.config import Settings
|
|
|
|
_STRONG_SECRET = "x" * 48
|
|
_PROD_DB = "postgresql+asyncpg://dadao_prod_app:pw@192.168.8.160:8432/dadaotask_prod"
|
|
|
|
|
|
def _prod(**overrides: object) -> Settings:
|
|
values: dict[str, object] = {
|
|
"env": "prod",
|
|
"jwt_secret": _STRONG_SECRET,
|
|
"database_url": _PROD_DB,
|
|
"sms_provider": "aliyun",
|
|
}
|
|
values.update(overrides)
|
|
# _env_file=None：本机 backend/.env 里有 EMAIL_CODE_DEV_ECHO=true（跑 e2e 用的），
|
|
# 那条在生产是硬错误；这里要验的是 prod 自己的规则，不能被开发机的 .env 带偏。
|
|
return Settings(_env_file=None, **values)
|
|
|
|
|
|
def test_prod_rejects_dev_sms_when_login_enabled() -> None:
|
|
with pytest.raises(RuntimeError, match="SMS_LOGIN_ENABLED"):
|
|
_prod(sms_provider="dev", sms_login_enabled=True).validate_runtime()
|
|
|
|
|
|
def test_prod_is_fine_with_sms_login_closed() -> None:
|
|
# 默认就是这条：短信登录关着，provider 配成 dev 也不会有人能用它进来
|
|
_prod(sms_provider="dev", sms_login_enabled=False).validate_runtime()
|
|
|
|
|
|
def test_prod_allows_real_sms_provider_when_enabled() -> None:
|
|
_prod(sms_provider="aliyun", sms_login_enabled=True).validate_runtime()
|
|
|
|
|
|
def test_prod_still_rejects_weak_jwt_secret() -> None:
|
|
with pytest.raises(RuntimeError, match="JWT_SECRET"):
|
|
_prod(jwt_secret="short").validate_runtime()
|