From cd6e235848ec4d4ec1831d4a69752de41ea68f66 Mon Sep 17 00:00:00 2001 From: HenryDu8133 <813367384@qq.com> Date: Wed, 30 Sep 2026 23:49:40 +0800 Subject: [PATCH] =?UTF-8?q?fix(users):=20=E7=8A=B6=E6=80=81=E5=A4=A7?= =?UTF-8?q?=E5=B0=8F=E5=86=99=E4=B8=8D=E6=95=8F=E6=84=9F=20+=20=E5=BD=BB?= =?UTF-8?q?=E5=BA=95=E4=BF=AE=E6=8E=89=E3=80=8C=E6=97=A0=E6=B3=95=E7=BC=96?= =?UTF-8?q?=E8=BE=91=E5=85=B6=E5=AE=83=E7=94=A8=E6=88=B7=E3=80=8D?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 站长反馈仍然无法编辑其它用户。上一版只把**小程序端**的状态改成小写， 但如果小程序没重新编译（或用的还是旧版本），发上来的仍是 `ACTIVE`， 服务端 `['active','withdrawn','graduate'].includes(status)` 依旧不通过 → 400 「用户状态只能是 active, withdrawn 或 graduate」——编辑任何用户都会失败。 这版把容忍度放到**服务端**，与客户端版本解耦： - 新增 server/utils/user-status.ts： · USER_STATUSES = ['active','withdrawn','graduate'] · normalizeUserStatus() 大小写不敏感（'ACTIVE' / ' Active ' → 'active'），认不出返回 null · USER_STATUS_RULE_MESSAGE 统一文案 - [id].put.ts：状态改用归一化值；另修正"自己只能改姓名"里的状态比较也忽略大小写 - [id]/status.put.ts、batch-status.put.ts：同样归一化； 并修掉一个我自己引入的隐患（把归一化结果赋回 const 变量 → TypeError）， batch-status 之前被误写成不存在的 statusesRaw，一并改正 效果：无论小程序是否重新编译，管理员都能编辑其它用户（含历史拼音账号只改年级）。 --- server/api/admin/users/[id].put.ts | 15 ++++++++---- server/api/admin/users/[id]/status.put.ts | 10 ++++---- server/api/admin/users/batch-status.put.ts | 8 ++++--- server/utils/user-status.ts | 27 ++++++++++++++++++++++ 4 files changed, 48 insertions(+), 12 deletions(-) create mode 100644 server/utils/user-status.ts diff --git a/server/api/admin/users/[id].put.ts b/server/api/admin/users/[id].put.ts index f65cf5b..f3ad6ac 100644 --- a/server/api/admin/users/[id].put.ts +++ b/server/api/admin/users/[id].put.ts @@ -5,6 +5,7 @@ import { users, userStatusLogs } from '~/drizzle/schema' import { eq } from 'drizzle-orm' import { updateUserPassword } from '~~/server/services/userService' import { getRoleLevel, canChangeRoleOf, canResetPasswordFor } from '~~/server/utils/permissions' +import { normalizeUserStatus, USER_STATUS_RULE_MESSAGE } from '~~/server/utils/user-status' export default defineEventHandler(async (event) => { try { @@ -189,11 +190,15 @@ export default defineEventHandler(async (event) => { }) } - // 验证status字段的有效性 - if (status && !['active', 'withdrawn', 'graduate'].includes(status)) { + /* + * 状态：**大小写不敏感**（客户端历史版本发的是大写的 ACTIVE）。 + * 认不出来才报错 —— 否则管理员会遇到"编辑任何用户都 400"（已踩过）。 + */ + const nextStatus = status === undefined || status === null ? null : normalizeUserStatus(status) + if (status !== undefined && status !== null && !nextStatus) { throw createError({ statusCode: 400, - statusMessage: '用户状态只能是 active, withdrawn 或 graduate' + statusMessage: USER_STATUS_RULE_MESSAGE }) } @@ -207,8 +212,8 @@ export default defineEventHandler(async (event) => { } // 如果提供了status，则更新状态相关字段 - if (status && status !== existingUser[0].status) { - updateData.status = status + if (nextStatus && nextStatus !== existingUser[0].status) { + updateData.status = nextStatus updateData.statusChangedAt = new Date() updateData.statusChangedBy = user.id } diff --git a/server/api/admin/users/[id]/status.put.ts b/server/api/admin/users/[id]/status.put.ts index c9137a2..17ee3bf 100644 --- a/server/api/admin/users/[id]/status.put.ts +++ b/server/api/admin/users/[id]/status.put.ts @@ -1,4 +1,5 @@ import { createError, defineEventHandler, getRouterParam, readBody } from 'h3' +import { normalizeUserStatus, USER_STATUS_RULE_MESSAGE } from '~~/server/utils/user-status' import { db } from '~/drizzle/db' import { users, userStatusLogs } from '~/drizzle/schema' import { eq } from 'drizzle-orm' @@ -18,13 +19,14 @@ export default defineEventHandler(async (event) => { const userId = getRouterParam(event, 'id') const body = await readBody(event) - const { status, reason } = body + const { status: rawStatus, reason } = body - // 验证必填字段 - if (!status || !['active', 'withdrawn', 'graduate'].includes(status)) { + // 验证必填字段；状态大小写不敏感（客户端历史版本发大写 ACTIVE 也能用） + const status = normalizeUserStatus(rawStatus) + if (!status) { throw createError({ statusCode: 400, - statusMessage: '状态必须为 active, withdrawn 或 graduate' + statusMessage: USER_STATUS_RULE_MESSAGE }) } diff --git a/server/api/admin/users/batch-status.put.ts b/server/api/admin/users/batch-status.put.ts index 6e06a39..3091ebd 100644 --- a/server/api/admin/users/batch-status.put.ts +++ b/server/api/admin/users/batch-status.put.ts @@ -17,7 +17,7 @@ export default defineEventHandler(async (event) => { } const body = await readBody(event) - const { userIds, status, reason } = body + const { userIds, status: rawStatus, reason } = body // 验证必填字段 if (!Array.isArray(userIds) || userIds.length === 0) { @@ -27,10 +27,12 @@ export default defineEventHandler(async (event) => { }) } - if (!status || !['active', 'withdrawn', 'graduate'].includes(status)) { + // 状态：大小写不敏感（客户端历史版本发的是大写 ACTIVE） + const status = normalizeUserStatus(rawStatus) + if (!status) { throw createError({ statusCode: 400, - statusMessage: '状态必须为 active, withdrawn 或 graduate' + statusMessage: USER_STATUS_RULE_MESSAGE }) } diff --git a/server/utils/user-status.ts b/server/utils/user-status.ts new file mode 100644 index 0000000..47761f8 --- /dev/null +++ b/server/utils/user-status.ts @@ -0,0 +1,27 @@ +import { eq } from 'drizzle-orm' + +/** + * 用户状态（`users.status`）的唯一口径。 + * + * ⚠️ 这里踩过一次真实的坑：小程序端一直发**大写** `ACTIVE`， + * 而服务端与 DB 用的是**小写** `active` / `withdrawn` / `graduate`（登录、2FA、WebAuthn + * 等地方都按小写判断）。于是管理员在用户管理里**编辑任何用户**都会被 + * 「用户状态只能是 active, withdrawn 或 graduate」拒绝 —— 看起来像"编辑功能坏了"。 + * + * 所以状态统一走这里的 `normalizeUserStatus()`：**大小写不敏感**， + * 认不出来的值返回 null，由调用方决定是报错还是忽略（创建用户时忽略、编辑时可报错）。 + */ +export const USER_STATUSES = ['active', 'withdrawn', 'graduate'] as const + +export type UserStatus = (typeof USER_STATUSES)[number] + +/** 归一化：`ACTIVE` / `active` / ` Active ` → `active`；其它 → null */ +export function normalizeUserStatus(value: unknown): UserStatus | null { + const s = String(value ?? '') + .trim() + .toLowerCase() + return (USER_STATUSES as readonly string[]).includes(s) ? (s as UserStatus) : null +} + +/** 统一的错误文案（列出允许值，别让调用方各写各的） */ +export const USER_STATUS_RULE_MESSAGE = '用户状态只能是 active、withdrawn 或 graduate'