fix(audit): 刷新令牌记录不再显示「未认证」
问题:审计列表里 POST /auth/refresh 的操作人恒为「未认证」。
根因:本接口刻意不挂 get_current_user —— 能用到这里,正是因为 access token
已过期、请求里没有 Authorization 头,JWT 依赖不执行,request.state 里
从未写入操作人。而"谁在何时尝试刷新"恰恰是该留痕的信息。
修复:
- core/security.py 新增 peek_token_identity(),与 decode_token 的唯一区别是
关闭过期校验(刷新场景令牌本就过期,若因过期解不出来还是会漏记)。
签名校验照常进行,伪造令牌解不出任何东西。
⚠️ docstring 中明确:该函数只许用于写审计字段,鉴权一律走 get_current_user
- refresh 接口解码 refresh token 取得 sub/username/display_name/role 写入 state。
refresh token 的载荷与 access token 完全一致,只有 type 字段不同。
附带效果:活动打点读的正是 request.state.audit_user,修复后刷新请求也会
被计为一次活动 —— 语义正确(会刷新说明用户正在使用)。
验证(7/7):正常刷新记到中文人名与角色;过期令牌虽被拒 401 但仍能记到人;
伪造令牌不认人、显示未认证。历史记录不追溯。
This commit is contained in:
@ -7,6 +7,7 @@ from app.schemas.user import (
|
||||
RefreshResponse,
|
||||
UserResponse,
|
||||
)
|
||||
from app.core.security import peek_token_identity
|
||||
from app.services.auth_service import login, refresh_access_token, get_current_user
|
||||
|
||||
router = APIRouter(prefix="/auth", tags=["认证"])
|
||||
@ -34,8 +35,19 @@ def login_endpoint(data: LoginRequest, request: Request):
|
||||
|
||||
|
||||
@router.post("/refresh", response_model=RefreshResponse)
|
||||
def refresh_endpoint(data: RefreshRequest):
|
||||
def refresh_endpoint(data: RefreshRequest, request: Request):
|
||||
"""刷新 Access Token — 使用 Refresh Token 换取新的 Access Token"""
|
||||
# 本接口刻意不挂 get_current_user:能用到这里,正是因为 access token 已经
|
||||
# 过期/缺失,请求里没有 Authorization 头,JWT 依赖不会执行 → 审计拿不到操作人,
|
||||
# 记录只能显示「未认证」。
|
||||
# 但 refresh token 里本来就带着完整身份(sub/username/display_name/role),
|
||||
# 解出来写进 state,审计才能记到人 —— 而"谁在何时尝试刷新"正是要留痕的。
|
||||
# 注意 peek 只用于审计标注,鉴权判断一律走 get_current_user。
|
||||
identity = peek_token_identity(data.refresh_token)
|
||||
if identity:
|
||||
request.state.audit_user = identity.get("username") or identity.get("sub")
|
||||
request.state.audit_display_name = identity.get("display_name") or ""
|
||||
request.state.audit_role = identity.get("role") or ""
|
||||
return refresh_access_token(data.refresh_token)
|
||||
|
||||
|
||||
|
||||
@ -39,6 +39,28 @@ def decode_token(token: str) -> dict:
|
||||
return jwt.decode(token, settings.SECRET_KEY, algorithms=[ALGORITHM])
|
||||
|
||||
|
||||
def peek_token_identity(token: str) -> dict | None:
|
||||
"""读出令牌里的用户身份 —— **仅供审计标注,绝不可用于授权**。
|
||||
|
||||
与 decode_token 的唯一区别:**关闭过期校验**。
|
||||
|
||||
为什么需要它:刷新令牌接口正是"access token 过期了才来"的场景,
|
||||
请求里不带 Authorization 头,JWT 依赖根本不执行,审计只能记成
|
||||
「未认证」—— 而"谁在什么时候尝试刷新"恰恰是该留痕的信息。
|
||||
签名校验照常进行,伪造的令牌解不出任何东西。
|
||||
|
||||
⚠️ 返回值只允许写进 request.state 的审计字段;
|
||||
任何鉴权判断一律走 get_current_user,不要用本函数。
|
||||
"""
|
||||
try:
|
||||
return jwt.decode(
|
||||
token, settings.SECRET_KEY, algorithms=[ALGORITHM],
|
||||
options={"verify_exp": False},
|
||||
)
|
||||
except JWTError:
|
||||
return None
|
||||
|
||||
|
||||
def verify_password(plain_password: str, hashed_password: str) -> bool:
|
||||
"""验证明文密码 vs 哈希密码"""
|
||||
return pwd_context.verify(plain_password, hashed_password)
|
||||
|
||||
Reference in New Issue
Block a user