3.3 KiB

Raw Permalink Blame History

修复短信验证码测试绕过逻辑

日期: 2026-01-27
类型: Bug 修复
影响范围: 短信验证码服务

问题描述

手机号登录接口 /api/v1/auth/login/phone 使用测试验证码 6666 时仍然提示"验证码不存在或已过期"。

根本原因

SmsService.verify_code() 方法缺少测试验证码绕过逻辑，直接查询数据库导致：

如果数据库中没有 6666 的记录 → 提示"验证码不存在或已过期"
如果记录已过期 → 同样提示过期

而 UserService.bind_phone() 方法中有正确的测试验证码逻辑，导致行为不一致。

解决方案

在 SmsService.verify_code() 方法开头添加测试验证码绕过逻辑：

async def verify_code(
    self,
    phone: str,
    country_code: str,
    code: str,
    purpose: SmsPurpose = SmsPurpose.LOGIN
) -> bool:
    """验证验证码"""
    
    # 测试验证码绕过（开发阶段）
    if code == "6666":
        logger.info(f"使用测试验证码: phone={phone}, purpose={purpose.to_string()}")
        return True
    
    # 正常验证流程...
    verification_code = await self.repository.get_valid_code(...)
    ...

修改文件

server/app/services/sms_service.py
- 在 verify_code() 方法开头添加测试验证码判断
- 记录测试验证码使用日志

测试验证

测试用例 1：使用测试验证码登录

curl -X POST http://localhost:8000/api/v1/auth/login/phone \
  -H "Content-Type: application/json" \
  -d '{
    "phone": "13800138000",
    "countryCode": "+86",
    "code": "6666"
  }'

预期结果：

✅ 登录成功
✅ 返回用户信息和 token
✅ 日志显示"使用测试验证码"

测试用例 2：使用真实验证码登录

# 1. 发送验证码
curl -X POST http://localhost:8000/api/v1/auth/sms/send \
  -H "Content-Type: application/json" \
  -d '{
    "phone": "13800138000",
    "countryCode": "+86",
    "purpose": "login"
  }'

# 2. 使用真实验证码登录
curl -X POST http://localhost:8000/api/v1/auth/login/phone \
  -H "Content-Type: application/json" \
  -d '{
    "phone": "13800138000",
    "countryCode": "+86",
    "code": "123456"
  }'