fix(test): 修复JWT篡改签名测试flaky - 改签名中间字符而非最后一位 #770

Merged

xiaoxia merged 1 commits from fix/jwt-tampered-test-flaky into develop 2026-07-23 21:52:01 +08:00

Conversation 2 Commits 1 Files Changed 1

+9 -2

xiaoxia commented 2026-07-23 19:43:00 +08:00

Owner

Copy Link

Copy Source

问题

test_verify_token_tampered_signature 是flaky test，有时候过有时候挂。

根因：测试只篡改JWT的最后一个字符，但JWT是base64url编码，最后一位可能是padding或者不影响签名验证的位，导致篡改无效，测试假阴性。

修复

改为篡改签名部分中间的字符，确保签名验证一定会失败，测试稳定。

JWT格式：header.payload.signature，找到签名部分（最后一段中间字符进行篡改。

改动

• tests/unit/test_auth_service.py - 1个测试用例

## 问题 `test_verify_token_tampered_signature` 是flaky test，有时候过有时候挂。 **根因**：测试只篡改JWT的**最后一个字符**，但JWT是base64url编码，最后一位可能是padding或者不影响签名验证的位，导致篡改无效，测试假阴性。 ## 修复 改为篡改**签名部分中间的字符**，确保签名验证一定会失败，测试稳定。 JWT格式：`header.payload.signature`，找到签名部分（最后一段中间字符进行篡改。 ## 改动 - `tests/unit/test_auth_service.py` - 1个测试用例

xiaoxia added 1 commit 2026-07-23 21:39:11 +08:00

fix(test): 修复JWT篡改签名测试flaky - 改签名中间字符而非最后一位 78c03e157b

xiaoxia force-pushed fix/jwt-tampered-test-flaky from 0965faadf5 to 78c03e157b 2026-07-23 21:39:11 +08:00 Compare

auto-approve-bot commented 2026-07-23 21:46:19 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #770

⚠️ 问题（0个需要修改）

无

💡 建议（0个可选）

无

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-23 13:46:19 | 模型:}

## 代码审查结果 - PR #770 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（0个可选） 无 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-23 13:46:19 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 678d3503a7 into develop 2026-07-23 21:52:01 +08:00

gitea-actions bot commented 2026-07-23 21:52:11 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #770 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #770 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#770