fix(api): 修复验证码repository datetime时区问题 - naive转aware #755

Merged

xiaoxia merged 1 commits from fix/verification-code-datetime-timezone into develop 2026-07-23 17:32:51 +08:00

Conversation 2 Commits 1 Files Changed 1

+14 -3

xiaoxia commented 2026-07-23 13:21:27 +08:00

Owner

Copy Link

Copy Source

No description provided.

xiaoxia added 1 commit 2026-07-23 13:21:27 +08:00

fix(api): 修复验证码repository datetime时区问题 - 数据库naive转aware ... b701182f6b

SQLAlchemy 从数据库读出的 DateTime 是 naive（不带时区），
但领域模型 VerificationCode 用的是 aware datetime（带 timezone.utc），
直接比较会报 "can't compare offset-naive and offset-aware datetimes"。

修复：_to_entity 时给所有 datetime 字段补 timezone.utc。

影响：send-verification-code 限流检查、bind-contact 验证码校验（is_expired）

auto-approve-bot commented 2026-07-23 13:26:26 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #755

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

无

💡 建议（1个可选）

1. verification_code_repository.py 第81行：时区假设风险
   • 代码使用 replace(tzinfo=timezone.utc) 强制将数据库读出的时间标记为 UTC。这假设数据库中存储的时间本身就是 UTC 时间（只是没带时区信息）。如果数据库实际存储的是本地服务器时间（如 CST），此操作会导致时间语义错误（时间值不变但时区标签变了，导致实际时间偏移）。
   • 建议：确认数据库字段配置及写入逻辑，确保存储的确实是 UTC 时间。如果数据库可能存储本地时间，应结合 astimezone 进行转换。

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能无明显问题

---

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

## 代码审查结果 - PR #755 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（1个可选） 1. **verification_code_repository.py 第81行**：时区假设风险 - 代码使用 `replace(tzinfo=timezone.utc)` 强制将数据库读出的时间标记为 UTC。这假设数据库中存储的时间本身就是 UTC 时间（只是没带时区信息）。如果数据库实际存储的是本地服务器时间（如 CST），此操作会导致时间语义错误（时间值不变但时区标签变了，导致实际时间偏移）。 - 建议：确认数据库字段配置及写入逻辑，确保存储的确实是 UTC 时间。如果数据库可能存储本地时间，应结合 `astimezone` 进行转换。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能无明显问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-23 05:26:26 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit b5ad67830e into develop 2026-07-23 17:32:51 +08:00

xiaoxia deleted branch fix/verification-code-datetime-timezone 2026-07-23 17:32:51 +08:00

gitea-actions bot commented 2026-07-23 17:38:21 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #755 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 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#755