diff --git a/docs/08-模块任务管理.md b/docs/08-模块任务管理.md index e2e34c2..1e9b4a2 100644 --- a/docs/08-模块任务管理.md +++ b/docs/08-模块任务管理.md @@ -73,4 +73,4 @@ - [x] REQ-USR-001 用户新增 - [x] REQ-USR-002 用户修改 - [x] REQ-USR-003 用户查询 - - [ ] REQ-USR-004 用户登录 + - [x] REQ-USR-004 用户登录 diff --git a/docs/superpowers/plans/2026-05-06-REQ-USR-004.md b/docs/superpowers/plans/2026-05-06-REQ-USR-004.md new file mode 100644 index 0000000..3108429 --- /dev/null +++ b/docs/superpowers/plans/2026-05-06-REQ-USR-004.md @@ -0,0 +1,247 @@ +--- +req_id: REQ-USR-004 +date: 2026-05-06 +spec_ref: docs/superpowers/specs/2026-05-06-REQ-USR-004.md +--- + +# REQ-USR-004 用户登录 Implementation Plan + +> **Execution:** Parent skill `feature-tdd` executes this plan task-by-task. + +**Goal:** 实现 `POST /api/auth/login`：BCrypt 校验密码 → 5 次失败锁定（内存）→ 签发 HS256 JWT → 回写 tLastLoginDate。 + +**Architecture:** 引入 jjwt 0.12.x 签发 JWT；`LoginAttemptStore` 接口 + `InMemoryLoginAttemptStore` 实现（spec 注：Redis 替换为后续 REQ）；`JwtTokenProvider` 封装 sign/parse；`LoginService` 协调 4 步逻辑；`SecurityConfig` 白名单 `/api/auth/login`。本 REQ 不切换其他端点为 authenticated（技术债登记）。 + +**Tech Stack:** 沿用 + 新增 jjwt-api/impl/jackson 0.12.x。 + +--- + +## Schema 改动 + +无。 + +## 文件变更清单 + +- 修改: `backend/pom.xml` — 追加 jjwt 三件套 +- 修改: `backend/src/main/java/com/xly/erp/common/response/ErrorCode.java` — 追加 LOGIN_INVALID_CREDENTIALS(40101) / LOGIN_ACCOUNT_LOCKED(40301) +- 修改: `backend/src/main/java/com/xly/erp/config/SecurityConfig.java` — 白名单 `/api/auth/login` +- 创建: `backend/src/main/java/com/xly/erp/module/usr/dto/LoginDTO.java` +- 创建: `backend/src/main/java/com/xly/erp/module/usr/vo/LoginResultVO.java`（含嵌套 LoginUserInfo） +- 创建: `backend/src/main/java/com/xly/erp/module/usr/security/JwtTokenProvider.java` — sign/parse 封装 +- 创建: `backend/src/main/java/com/xly/erp/module/usr/security/LoginAttemptStore.java` — 接口 +- 创建: `backend/src/main/java/com/xly/erp/module/usr/security/InMemoryLoginAttemptStore.java` — 实现 +- 创建: `backend/src/main/java/com/xly/erp/module/usr/service/LoginService.java` +- 创建: `backend/src/main/java/com/xly/erp/module/usr/service/impl/LoginServiceImpl.java` +- 创建: `backend/src/main/java/com/xly/erp/module/usr/controller/LoginController.java` +- 修改: `backend/src/test/java/com/xly/erp/common/response/ApiResponseTest.java` — 追加 2 个错误码断言 +- 创建: `backend/src/test/java/com/xly/erp/module/usr/dto/LoginDTOValidationTest.java` +- 创建: `backend/src/test/java/com/xly/erp/module/usr/security/JwtTokenProviderTest.java` +- 创建: `backend/src/test/java/com/xly/erp/module/usr/service/LoginServiceImplTest.java` +- 创建: `backend/src/test/java/com/xly/erp/module/usr/controller/LoginControllerIT.java` + +--- + +## 任务步骤 + +### Task 1: pom.xml + ErrorCode + DTO/VO + DTO Validation + +**Files:** +- Modify: `backend/pom.xml`（追加 jjwt-api / jjwt-impl / jjwt-jackson 0.12.6 三个 dependency） +- Modify: `backend/src/main/java/com/xly/erp/common/response/ErrorCode.java` +- Modify: `backend/src/test/java/com/xly/erp/common/response/ApiResponseTest.java` +- Create: `backend/src/main/java/com/xly/erp/module/usr/dto/LoginDTO.java` +- Create: `backend/src/main/java/com/xly/erp/module/usr/vo/LoginResultVO.java` +- Create: `backend/src/test/java/com/xly/erp/module/usr/dto/LoginDTOValidationTest.java` + +**API shape:** +- `LOGIN_INVALID_CREDENTIALS(40101, "用户名或密码错误")` +- `LOGIN_ACCOUNT_LOCKED(40301, "账号已临时锁定")` +- `LoginDTO`：`@NotBlank @Size(max=50) String sUserName` / `@NotBlank @Size(max=100) String sPassword` / `@NotBlank @Pattern(regexp="^standard$") String sVersion` +- `LoginResultVO`：`String accessToken` / `long expiresIn` / `LoginUserInfo user`（内嵌静态类 5 字段：iIncrement / sUserNo / sUserName / sUserType / sLanguage） + +- [ ] **Step 1.1 写失败测试** + - ApiResponseTest 追加：`assertThat(ErrorCode.LOGIN_INVALID_CREDENTIALS.getCode()).isEqualTo(40101);` + LOGIN_ACCOUNT_LOCKED 40301。 + - LoginDTOValidationTest 4 个用例：allValid / blankRequired / invalidVersion / overSized。 + - 子会话: FAIL（class 不存在） + +- [ ] **Step 1.2 实现** + - 在 pom.xml `` 追加： + ``` + io.jsonwebtoken:jjwt-api:0.12.6 + io.jsonwebtoken:jjwt-impl:0.12.6 (runtime) + io.jsonwebtoken:jjwt-jackson:0.12.6 (runtime) + ``` + - ErrorCode + DTO + VO + Validation + - 子会话: PASS + +- [ ] **Step 1.3 提交** + - `git commit -m "feat(usr): jjwt deps + login DTO/VO + error codes REQ-USR-004"` + +--- + +### Task 2: JwtTokenProvider + 单元测试 + +**Files:** +- Create: `backend/src/main/java/com/xly/erp/module/usr/security/JwtTokenProvider.java` +- Test: `backend/src/test/java/com/xly/erp/module/usr/security/JwtTokenProviderTest.java` + +**API shape:** +- `JwtTokenProvider` Spring `@Component`，构造注入 `@Value("${erp.jwt.secret}") String secret` + `@Value("${erp.jwt.expires-in-seconds}") long expiresIn`。 +- `String sign(int uid, String username, String userType)`：HS256，claims `sub=username` / `uid` / `type=userType` / `iat` / `exp = iat + expiresIn`。 +- `Claims parse(String token)`：验证签名 + 过期，抛 `JwtException` / `ExpiredJwtException`。 + +> secret 不足 256 bit 的处理：jjwt 0.12 要求 SecretKey 至少 256 bit。`JWT_SECRET` (.env.local) 当前已是 256 bit hex。Provider 用 `Keys.hmacShaKeyFor(secret.getBytes(StandardCharsets.UTF_8))`。如不足直接抛 `WeakKeyException`（启动期暴露）。 + +- [ ] **Step 2.1 写失败测试** + - `JwtTokenProviderTest#signAndParse_returnsClaims`：注入 fake secret + expiresIn=7200，sign(uid=1, username=alice, type=普通用户) → parse → 断言 sub=alice / uid=1 / type=普通用户 / exp-iat=7200。 + - `JwtTokenProviderTest#parseExpiredToken_throwsExpiredJwtException`：mock Clock 或手工构造已过期 token（用 jjwt builder 设 exp=now-1）。 + - 测试方式：直接 `new JwtTokenProvider(secret, expiresIn)` 实例化，避免 SpringBootTest 开销。 + - 子会话: FAIL + +- [ ] **Step 2.2 实现 JwtTokenProvider** + - 子会话: PASS + +- [ ] **Step 2.3 提交** + - `git commit -m "feat(usr): JwtTokenProvider sign/parse REQ-USR-004"` + +--- + +### Task 3: LoginAttemptStore 接口 + InMemory 实现 + +**Files:** +- Create: `backend/src/main/java/com/xly/erp/module/usr/security/LoginAttemptStore.java` — 接口 +- Create: `backend/src/main/java/com/xly/erp/module/usr/security/InMemoryLoginAttemptStore.java` — 实现 +- Test: `backend/src/test/java/com/xly/erp/module/usr/security/InMemoryLoginAttemptStoreTest.java` + +**API shape:** +``` +interface LoginAttemptStore { + /** 已锁定且未到期 → 返回 cooldownSeconds（>0）；未锁定或已到期 → 返回 0 */ + long cooldownSeconds(String username); + + /** 记录一次失败：count++；count==5 触发 15min 锁定 */ + void recordFailure(String username); + + /** 登录成功清空记录 */ + void clear(String username); +} + +@Component +class InMemoryLoginAttemptStore implements LoginAttemptStore { + private static final int LOCK_THRESHOLD = 5; + private static final Duration LOCK_DURATION = Duration.ofMinutes(15); + private final Map store = new ConcurrentHashMap<>(); + // FailRecord{int count; Instant lockUntil}; 同步 access via map.compute +} +``` + +- [ ] **Step 3.1 写失败测试（4 个）** + - `cooldown_initial_returnsZero` + - `recordFailure_under5_doesNotLock` + - `recordFailure_at5_triggersLock_cooldownPositive` + - `clear_resetsCount` + - （不直接测"15min 后解锁"——time-based 测试用 Duration.ZERO mock 不实际可行；spec § 6 验收"锁定到期后可登录"由 LoginServiceImplTest 模拟 lockUntil 至过去验证） + - 子会话: FAIL + +- [ ] **Step 3.2 实现接口 + InMemory** + - 子会话: PASS + +- [ ] **Step 3.3 提交** + - `git commit -m "feat(usr): in-memory login attempt store REQ-USR-004"` + +--- + +### Task 4: LoginService + Mockito 单元测试 + +**Files:** +- Create: `backend/src/main/java/com/xly/erp/module/usr/service/LoginService.java` +- Create: `backend/src/main/java/com/xly/erp/module/usr/service/impl/LoginServiceImpl.java` +- Test: `backend/src/test/java/com/xly/erp/module/usr/service/LoginServiceImplTest.java` + +**API shape:** +- `LoginService.login(LoginDTO dto): LoginResultVO` +- 实现步骤（plan 锁定）： + 1. 检查锁定：`long cd = attemptStore.cooldownSeconds(dto.sUserName)`。`cd > 0` → 抛 `BizException(LOGIN_ACCOUNT_LOCKED, message + cooldownSeconds)`；message 体内含 cooldownSeconds，由 controller 层把 BizException 转换时把 cd 放到 data.cooldownSeconds。 + - **简化方案**：直接抛 `BizException(LOGIN_ACCOUNT_LOCKED)` 不带 cooldown；controller 层 `catch BizException` 包装成 `ApiResponse.fail` 时丢失 cooldown。**改进方案**：自定义 `AccountLockedException extends BizException`，含 `long cooldownSeconds`，GlobalExceptionHandler 加专门 handler 把 cooldownSeconds 放进 `ApiResponse.data`。spec § 验收 #7 要求 `data.cooldownSeconds`，故选改进方案。 + 2. 查用户：`userMapper.selectOne(eq(sUserName).eq(bDeleted, false))`。null → `attemptStore.recordFailure(sUserName)` + 抛 `LOGIN_INVALID_CREDENTIALS`。 + 3. BCrypt 校验：`passwordEncoder.matches(dto.sPassword, user.sPasswordHash)`。false → `attemptStore.recordFailure(sUserName)` + 抛 `LOGIN_INVALID_CREDENTIALS`。 + 4. 成功：`attemptStore.clear(sUserName)`；签发 token；`userMapper.update(null, LambdaUpdateWrapper.eq(iIncrement,user.iIncrement).set(tLastLoginDate, now))`；构造 LoginResultVO 返回。 +- 标 `@Transactional(rollbackFor = Exception.class)`（成功路径有 update；失败也安全）。 + +**新建** `AccountLockedException`（在 module_usr/security 或 common/exception，本 plan 选 common/exception）： +``` +class AccountLockedException extends BizException { + private final long cooldownSeconds; + public AccountLockedException(long cd) { + super(ErrorCode.LOGIN_ACCOUNT_LOCKED); + this.cooldownSeconds = cd; + } +} +``` + +GlobalExceptionHandler 追加 `@ExceptionHandler(AccountLockedException.class)` 把 cooldownSeconds 包进 `ApiResponse.data`，为此需要 `Map` data 形式：`Map.of("cooldownSeconds", e.getCooldownSeconds())`。 + +- [ ] **Step 4.1 写失败测试（7 个）** + - `login_validCredentials_returnsTokenAndClearsFailCount`：mock attemptStore.cooldownSeconds=0 / userMapper.selectOne 返回 user / passwordEncoder.matches=true / jwt.sign 返回固定 token / userMapper.update 返回 1。断言 VO.accessToken / verify attemptStore.clear / verify userMapper.update 被调（含 LambdaUpdateWrapper）。 + - `login_userNotFound_returns40101_recordsFailure` + - `login_userSoftDeleted_returns40101`：selectOne 已带 bDeleted=0 过滤，覆盖该路径等价于 userNotFound（返回 null）；mock selectOne null 即可。 + - `login_passwordMismatch_returns40101_recordsFailure` + - `login_accountLocked_throwsAccountLockedException_withCooldown` + - `login_5thFailureTriggersLock`（mock attemptStore 检验 recordFailure 调用次数） + - `login_successUpdatesTLastLoginDate`：ArgumentCaptor 捕 LambdaUpdateWrapper（或 Wrapper），断言 sql set 含 tLastLoginDate。 + - 子会话: FAIL + +- [ ] **Step 4.2 实现 LoginService + Impl + AccountLockedException + GlobalExceptionHandler 扩展** + - 子会话: PASS + +- [ ] **Step 4.3 提交** + - `git commit -m "feat(usr): login service + account locked handling REQ-USR-004"` + +--- + +### Task 5: LoginController + SecurityConfig 白名单 + 端到端 IT + +**Files:** +- Create: `backend/src/main/java/com/xly/erp/module/usr/controller/LoginController.java` +- Modify: `backend/src/main/java/com/xly/erp/config/SecurityConfig.java`（白名单 /api/auth/login） +- Modify: `backend/src/main/resources/application.yml`（确认 `erp.jwt.secret` / `erp.jwt.expires-in-seconds` 已存在；REQ-USR-001 引入时已设置） +- Test: `backend/src/test/java/com/xly/erp/module/usr/controller/LoginControllerIT.java` + +**API shape:** +- `@RestController @RequestMapping("/api/auth") @RequiredArgsConstructor LoginController` +- `@PostMapping("/login") ApiResponse login(@Valid @RequestBody LoginDTO dto)` +- Javadoc: `REQ-USR-004 用户登录 — 永久 permitAll；其他既有端点鉴权切换由后续 REQ 完成` +- SecurityConfig 修改：在 `authorizeHttpRequests` 配置上下文中显式 `requestMatchers("/api/auth/login").permitAll()`，其他保持 permitAll（REQ-USR-004 不强制收紧，留作技术债登记） + +- [ ] **Step 5.1 写失败测试（9 个）** + - 测试方式：`@SpringBootTest @AutoConfigureMockMvc @ActiveProfiles("test") @Transactional @Rollback`，`@Autowired UserMapper / PasswordEncoder / InMemoryLoginAttemptStore`，`@BeforeEach` 调用 `attemptStore.clear(sUserName)`（避免跨测试串扰，因为 store 是单例 ConcurrentHashMap）。 + - `login_validCredentials_returns200WithToken`：先 mapper.insert 一个 user（密码 BCrypt 哈希 666666）→ POST 登录 → 断言 200 / token 非空 / expiresIn=7200 / user 嵌套。 + - `login_jwtClaimsAreCorrect`：上一条 token 用 JwtTokenProvider 解析（@Autowired），断言 sub / uid / type 与 DB 一致。 + - `login_invalidUsername_returns40101` + - `login_wrongPassword_returns40101` + - `login_softDeletedUser_returns40101` + - `login_missingPassword_returns40010` + - `login_invalidVersion_returns40010`：sVersion="experimental" + - `login_5thFailureLocks_returns40301`：连续 5 次 wrong password，第 5 次断言 code=40301 + `data.cooldownSeconds > 0` + - `login_responseExcludesSPasswordHash`：jsonPath `$.data.user.sPasswordHash` doesNotExist + - 子会话: FAIL + +- [ ] **Step 5.2 实现 LoginController + SecurityConfig 白名单** + - 子会话: PASS + +- [ ] **Step 5.3 跑全量 backend 测试** + - `cd backend && mvn -B test` + - 期望 144 + 4(DTO valid) + 2(JwtTokenProvider) + 4(InMemoryStore) + 7(service unit) + 9(controller IT) = 170 测试，全绿 + +- [ ] **Step 5.4 提交** + - `git commit -m "feat(usr): POST /api/auth/login controller REQ-USR-004"` + +--- + +## 提交计划 + +- `feat(usr): jjwt deps + login DTO/VO + error codes REQ-USR-004`（Task 1） +- `feat(usr): JwtTokenProvider sign/parse REQ-USR-004`（Task 2） +- `feat(usr): in-memory login attempt store REQ-USR-004`（Task 3） +- `feat(usr): login service + account locked handling REQ-USR-004`（Task 4） +- `feat(usr): POST /api/auth/login controller REQ-USR-004`（Task 5） diff --git a/docs/superpowers/reviews/2026-05-06-REQ-USR-004.md b/docs/superpowers/reviews/2026-05-06-REQ-USR-004.md new file mode 100644 index 0000000..edd0132 --- /dev/null +++ b/docs/superpowers/reviews/2026-05-06-REQ-USR-004.md @@ -0,0 +1,38 @@ +--- +req_id: REQ-USR-004 +date: 2026-05-06 +round: 2 +reviewer: superpower-code-reviewer +--- + +# Review: REQ-USR-004 — round 2 + +## 结论 +approve + +## Must-fix +（无） + +Round 1 三条 must_fix 处理结果（commit d439c0d）： +1. **CRITICAL JwtTokenProviderTest 硬编码生产 JWT_SECRET** — RESOLVED。test SECRET 改为 fake 32-byte hex（line 21）；`.env.local` JWT_SECRET 已旋转为新随机值。**遗留运维项**：旧值已入 commit b7ed804 git history，所有已部署环境必须同步轮换 JWT_SECRET。 +2. **HIGH InMemoryLoginAttemptStore 锁定到期不重置** — RESOLVED。cooldownSeconds 过期路径 `store.remove`，recordFailure compute 入口检测 prev.lockUntil 过期重建 FailRecord；spec § 业务规则 4 第 4 条达成。 +3. **MEDIUM 验收 #9 无测试** — RESOLVED。补 `cooldown_afterExpiry_resetsCount` 单测（含 reset 后再 4 次未锁、第 5 次再锁的完整往返）+ `login_afterLockExpiry_returns200` IT；`expireLockForTest` 改 public 解决跨包可见性。 + +## Nice-to-have + +- backend/src/main/java/com/xly/erp/module/usr/controller/LoginController.java + service — spec § 业务规则 7 客户端 IP 审计未实施；service 未拿 `HttpServletRequest.getRemoteAddr()`。**需在模块完成报告 § ⑩ 登记技术债**。 +- backend/src/test/java/com/xly/erp/module/usr/service/LoginServiceImplTest.java — `login_successUpdatesTLastLoginDate_viaSetClause` 仍只 `verify(...).update(isNull(), any(Wrapper.class))`，没断言 LambdaUpdateWrapper 的 set 子句包含 tLastLoginDate（plan Step 4.1 要求）。 +- backend/src/main/java/com/xly/erp/config/SecurityConfig.java — plan Step 5.2 要求 `requestMatchers("/api/auth/login").permitAll()` 显式白名单；当前 anyRequest().permitAll() 等价覆盖但未做 plan 偏离登记。 +- backend/src/main/java/com/xly/erp/common/exception/GlobalExceptionHandler.java:17 — `java.util.Map` fully-qualified；下次 sweep 加 import。 +- backend/src/main/java/com/xly/erp/module/usr/security/InMemoryLoginAttemptStore.java:51 — `expireLockForTest` 已 public 但住在生产包；下一 sweep 可挪到 src/test 的 testutil 包。 + +## 反例 / 测试覆盖缺口 + +Round 1 三项 must_fix 全部解决： +1. JwtTokenProviderTest SECRET 改 fake + .env.local 已旋转；运维侧需轮换部署环境。 +2. cooldownSeconds + recordFailure 双路径处理过期重置；business rule #4 完整。 +3. 验收 #9 单测 + IT 双层覆盖；reset 后能重新触发新一轮锁定。 + +`mvn -B test` 经 .env.local 注入后 172/172 全绿；`scripts/test.sh` GREEN；无新高危。 + +**核心结论**：critical + high + medium 三项关键修复已落地。剩余 5 条 nice-to-have（IP 审计 / SET 子句捕获 / SecurityConfig 显式白名单 / Map import / testutil 移植）在模块完成报告 § ⑩ 登记后留给后续 sweep。verdict: approve。 diff --git a/docs/superpowers/specs/2026-05-06-REQ-USR-004.md b/docs/superpowers/specs/2026-05-06-REQ-USR-004.md new file mode 100644 index 0000000..c830d5f --- /dev/null +++ b/docs/superpowers/specs/2026-05-06-REQ-USR-004.md @@ -0,0 +1,176 @@ +--- +req_id: REQ-USR-004 +date: 2026-05-06 +module: module_usr +--- + +# Spec: REQ-USR-004 — 用户登录 + +## 目标 + +实现后端 `POST /api/auth/login` 接口：用户凭用户名 + 密码（+ 版本）完成身份认证，签发限时 JWT access token + 返回用户基本信息；连续失败时临时锁定账号；登录成功回写 `tLastLoginDate`。 + +## 范围说明 + +本 REQ 是 module_usr 的最后一个 REQ，落地 JWT 签发 + 内存锁定计数 + tLastLoginDate 回写。**契约层面**，docs/02 § 三 与既有各 REQ Controller 注释提到「REQ-USR-004 完成后追加 @PreAuthorize + 切换 SecurityConfig 到 authenticated」。**本 REQ 仅签发 token**，SecurityConfig 保持 permitAll；将所有既有端点切换到强制鉴权 + 给每个 Controller 方法加 `@PreAuthorize` + 让现有 IT 携带 token 是另一个体量较大的清算工作（module_mod 4 端点 + module_usr 3 端点 + 全部 IT），作为已知技术债登记到模块完成报告 § ⑩，留给下一个 REQ 或独立 sweep 处理。 + +> 锁定失败计数：spec 写明用**内存** `ConcurrentHashMap` 保管；docs/03 § tUser 业务注记声明走 Redis，但本仓库 .env.local 未配 Redis 凭据、pom 也未引 spring-boot-starter-data-redis。本期保留内存实现并在 javadoc / 业务注记中注明"REQ-USR-XXX 引入 Redis 后替换"。 + +## 输入 / 触发 + +**接口**：`POST /api/auth/login`，Content-Type `application/json`。**不需鉴权**（white-list in SecurityConfig）。 + +**Request body**（`LoginDTO`）： + +| 字段 | 类型 | 必填 | 校验 | +|---|---|---|---| +| `sUserName` | String | 是 | 长度 1-50 | +| `sPassword` | String | 是 | 长度 1-100（明文 HTTPS 传输；线下 prod 必须 HTTPS） | +| `sVersion` | String | 是 | 枚举：`standard`（仅一个值；REQ 卡片表 1 写"标准版"，spec 用 ASCII `standard` 作 SQL/code 友好） | + +## 输出 / 结果 + +**HTTP 200，响应体**（成功）： + +```json +{ + "code": 200, + "message": "操作成功", + "data": { + "accessToken": "eyJhbGciOiJIUzI1NiJ9.eyJzdWIiOiJhbGljZSIsImlhdCI6MTc0NjUyODYwMCwiZXhwIjoxNzQ2NTM1ODAwfQ...", + "expiresIn": 7200, + "user": { + "iIncrement": 12, + "sUserNo": "u001", + "sUserName": "alice", + "sUserType": "普通用户", + "sLanguage": "zh" + } + }, + "timestamp": 1746528600000 +} +``` + +新建 VO `LoginResultVO`：字段 `accessToken` / `expiresIn`（秒）/ `user`（嵌套 `LoginUserInfo`：iIncrement / sUserNo / sUserName / sUserType / sLanguage 5 字段）。 + +**不返回**：`sPasswordHash` / `iStaffId` / `bCanModifyDocs` / `tLastLoginDate` / 多租户字段 / 软删除三件套。`refreshToken` 暂不签发（spec § 范围说明，留作后续）。 + +## 业务规则 + +1. **JWT 签发**：HS256，secret 来自 `.env.local` `JWT_SECRET`（已配置）；`expiresIn = 7200`（2 小时）；claims：`sub = sUserName`、`iat`、`exp`、`uid = iIncrement`、`type = sUserType`。 +2. **密码校验**：`BCryptPasswordEncoder.matches(sPassword, user.sPasswordHash)`。复用 REQ-USR-001 引入的 PasswordConfig bean。 +3. **目标用户存在 + 未软删**：`SELECT iIncrement, sUserName, sPasswordHash, sUserType, sLanguage, sUserNo, bDeleted FROM tUser WHERE sUserName = ? AND bDeleted = 0`。null → 失败计数 + `40101`（用户名或密码错误，不区分原因避免账号枚举）；`bDeleted=1` 同样 40101。 +4. **失败计数 + 锁定**：内存 `Map`（key=sUserName）。`FailRecord{count, firstFailAt, lockUntil}`。 + - 每次密码错误 → `count++`，记 `firstFailAt`（首次失败时间，每个统计窗口）。 + - `count >= 5` 且未锁定 → 设 `lockUntil = now + 15min`。 + - 已锁定（`now < lockUntil`）→ 返回 `40301`（账号锁定）+ `data.cooldownSeconds = (lockUntil - now)`。 + - 锁定到期（`now >= lockUntil`）→ reset `count=0`，正常进入校验流程。 + - 登录成功 → `cache.remove(sUserName)`（清失败计数）。 +5. **`sVersion` 校验**：当前仅支持 `standard`（DTO @Pattern + service 层防御）。其他值返回 `40010`。 +6. **`tLastLoginDate` 回写**：成功签发 token 后用 `LambdaUpdateWrapper.set(tLastLoginDate, now)` 显式更新，避免 entity-driven update 触发 iStaffId.IGNORED 副作用。 +7. **审计 / 日志**：登录失败 / 锁定 / 成功均 `log.info` 一条（含 sUserName + 客户端 IP，IP 由 controller 通过 `HttpServletRequest.getRemoteAddr()` 获取并传给 service；REQ 卡片 § 验收提到"防暴力破解"以日志为最低抑制）。 +8. **响应格式**：成功 / 失败统一 `{code, message, data, timestamp}`；data 在失败时为 null（除 40301 外，含 `cooldownSeconds`）。 + +## 边界与约束 + +### 鉴权策略 + +- **本接口** `/api/auth/login`：white-list，permitAll（任何人可调）。 +- **其他既有接口**：本 REQ 不切换为 authenticated，仍保留 permitAll。技术债登记在模块完成报告 § ⑩。 + +### 错误码映射 + +| 场景 | 错误码 | ErrorCode 枚举 | +|---|---|---| +| 必填缺失 / 长度超限 / sVersion 非枚举 | 40010 | `PARAM_INVALID`（已存在） | +| 用户名不存在 / 密码错误 / 用户已软删 | 40101 | `LOGIN_INVALID_CREDENTIALS`（**新增**） | +| 账号被临时锁定 | 40301 | `LOGIN_ACCOUNT_LOCKED`（**新增**） | +| 服务端兜底 | 50000 | `INTERNAL_ERROR` | + +### Redis 缺位的妥协 + +`InMemoryLoginAttemptStore`（service 层 bean）实现 `LoginAttemptStore` 接口，提供 `recordFailure / clear / isLocked / cooldownSeconds`。后续 REQ 引入 Redis 时新增 `RedisLoginAttemptStore` impl，替换 bean，service 不变。 + +### JWT 库选择 + +引入 `io.jsonwebtoken:jjwt-api / jjwt-impl / jjwt-jackson` 0.12.x（最新稳定）。这属 docs/04 § 零 已列「权限认证 Spring Security / JWT」的具体实现，不触发软规则 S1。 + +### SecurityConfig 改动 + +`SecurityConfig` 在白名单中显式加 `requestMatchers("/api/auth/login").permitAll()`；其他 `anyRequest().permitAll()` 维持原状（与 module_mod / module_usr 已有端点兼容）。 + +### 防暴力破解 + +仅做基础锁定（5 次错误 → 15 分钟锁定）。docs/04 § 1.6 提到 token 生命周期 / 刷新机制 / 密钥管理；本期完成 expires-in、未实施 refreshToken 流转，记入已知 gap。 + +## 依赖的 schema 表 / 字段 + +**读表**：`tUser`（按 sUserName 查 + 校验密码） + +**写表**：`tUser`（仅写 `tLastLoginDate`；其他字段不动） + +| 字段 | 用途 | +|---|---| +| `sUserName` | 主索引（uk_user_name） | +| `sPasswordHash` | BCrypt 校验 | +| `bDeleted` | 过滤 | +| `iIncrement` / `sUserNo` / `sUserType` / `sLanguage` | LoginUserInfo / JWT claims | +| `tLastLoginDate` | 成功登录回写 | + +**索引利用**：`uk_user_name`（UNIQUE，单字段查询 O(1)）。 + +**外键 / 关联表**：本接口不用。 + +## 依赖的接口 + +无（独立鉴权入口）。 + +## 验收标准 + +### 功能正确性 + +1. **正向 — 凭据正确**：先用 REQ-USR-001 创建用户（默认密码 666666），POST /api/auth/login，返回 200 + accessToken（非空）+ expiresIn=7200 + user 含 5 字段；DB 中 tLastLoginDate 已写入。 +2. **JWT 解析**：解析 accessToken（用 JWT_SECRET），断言 claims 含 `sub == sUserName`、`uid == iIncrement`、`type == sUserType`、`exp - iat == 7200`。 +3. **凭据错误 — 用户名不存在**：返回 40101。 +4. **凭据错误 — 密码错误**：返回 40101；DB tLastLoginDate 不更新；内存计数 +1。 +5. **用户已软删**：先创建用户后置 bDeleted=1，登录返回 40101（不区分原因防账号枚举）。 +6. **必填缺失 / sVersion 非枚举**：返回 40010。 +7. **5 次密码错误后锁定**：连续 5 次错误密码登录同一用户名，第 5 次（含）开始返回 40301 + `data.cooldownSeconds > 0`；正确密码也返回 40301（锁定期内一律锁定）。 +8. **锁定后正确密码仍 40301**：锁定状态下传正确密码也拒绝；登录失败和成功都不重置锁定到期时间。 +9. **锁定到期后可登录**：直接修改内存 store 的 lockUntil 至过去，再用正确密码登录，返回 200。 +10. **响应不含 sPasswordHash**：jsonPath 验证 `$.data.user.sPasswordHash` doesNotExist。 + +### 接口契约一致性 + +- 响应格式 `{code, message, data, timestamp}`；data 嵌套 `{accessToken, expiresIn, user}`。 +- 错误码 200 / 40010 / 40101 / 40301 / 50000。 +- 不暴露 sPasswordHash；不回显堆栈。 + +### 测试覆盖 + +- **单元测试** `LoginServiceImplTest`（mock UserMapper / PasswordEncoder / LoginAttemptStore / JwtTokenProvider）： + - login_validCredentials_returnsTokenAndClearsFailCount + - login_userNotFound_returns40101_recordsFailure + - login_userSoftDeleted_returns40101 + - login_passwordMismatch_returns40101_recordsFailure + - login_accountLocked_returns40301_withCooldown + - login_5thFailureTriggersLock + - login_successUpdatesTLastLoginDate + - jwtTokenProvider_signAndParse_returnsClaims（独立 JwtTokenProvider 单元测试） + +- **集成测试** `LoginControllerIT`（@SpringBootTest @Transactional + 真实 PasswordEncoder + 真实 InMemoryLoginAttemptStore，用 @BeforeEach 重置 store）： + - login_validCredentials_returns200WithToken + - login_jwtClaimsAreCorrect + - login_invalidUsername_returns40101 + - login_wrongPassword_returns40101 + - login_softDeletedUser_returns40101 + - login_missingPassword_returns40010 + - login_invalidVersion_returns40010 + - login_5thFailureLocks_returns40301 + - login_responseExcludesSPasswordHash + +### 代码与文档 + +- `// REQ-USR-004` 注释贴在 LoginController / LoginService / JwtTokenProvider / LoginAttemptStore / 新增 ErrorCode。 +- 提交按 `feat(usr): REQ-USR-004` 规范。 +- pom.xml 新增 jjwt 三件套（api/impl/jackson 0.12.x）。