xiaoxia/xiaoxia-saas
1
0
Fork 0
Code Issues 16 Pull Requests 12 Actions 34.2k Packages Projects Releases 7 Wiki Activity

[P2] URL 安全模块去中间层，Worker 统一引用 shared 层 #1835

New Issue
Closed
opened 2026-09-10 12:07:32 +08:00 by xiaoxia · 1 comment
xiaoxia commented 2026-09-10 12:07:32 +08:00
Owner
Copy Link
Copy Source

背景

URL 安全逻辑存在三层包装：packages/domain/url_security.py（核心） -> packages/shared/url_security.py（薄包装） -> apps/worker/video_processing/url_security.py（兼容层）。

现状

  • packages/domain/url_security.py：核心实现
  • packages/shared/url_security.py：薄包装，从 domain 重新导出
  • apps/worker/video_processing/url_security.py：向后兼容层，从 shared 重新导出
  • Worker 中 5 处仍通过 video_processing.url_security 间接引用

修复方案

  1. 将 apps/worker/ 中的 5 处 import 改为直接引用 packages.shared.url_security
  2. 删除 apps/worker/video_processing/url_security.py 兼容层
  3. 从 video_processing/__init__.py 中移除 url_security 导入

涉及文件

  • apps/worker/video_processing/pip_engine.py
  • apps/worker/video_processing/sticker_engine.py
  • apps/worker/video_processing/render_adapter.py
  • apps/worker/video_processing/__init__.py
  • apps/worker/worker_app/tasks/batch_download.py
  • apps/worker/worker_app/tasks/generation.py

验收标准

  • Worker 中无 video_processing.url_security 引用
  • 兼容层文件删除
  • CI 全绿，现有测试通过
## 背景 URL 安全逻辑存在三层包装：packages/domain/url_security.py（核心） -> packages/shared/url_security.py（薄包装） -> apps/worker/video_processing/url_security.py（兼容层）。 ## 现状 - packages/domain/url_security.py：核心实现 - packages/shared/url_security.py：薄包装，从 domain 重新导出 - apps/worker/video_processing/url_security.py：向后兼容层，从 shared 重新导出 - Worker 中 5 处仍通过 video_processing.url_security 间接引用 ## 修复方案 1. 将 apps/worker/ 中的 5 处 import 改为直接引用 packages.shared.url_security 2. 删除 apps/worker/video_processing/url_security.py 兼容层 3. 从 video_processing/__init__.py 中移除 url_security 导入 ## 涉及文件 - apps/worker/video_processing/pip_engine.py - apps/worker/video_processing/sticker_engine.py - apps/worker/video_processing/render_adapter.py - apps/worker/video_processing/__init__.py - apps/worker/worker_app/tasks/batch_download.py - apps/worker/worker_app/tasks/generation.py ## 验收标准 - Worker 中无 video_processing.url_security 引用 - 兼容层文件删除 - CI 全绿，现有测试通过
xiaoxia added the backendrefactorcleanup labels 2026-09-10 12:07:32 +08:00
xiaoxia commented 2026-09-15 18:00:52 +08:00
Author
Owner
Copy Link
Copy Source

本轮未推进，关闭。后续重构 URL 安全层时新开 Issue。

本轮未推进，关闭。后续重构 URL 安全层时新开 Issue。
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
architecture
架构设计
 backend
后端相关
 blocked
阻塞中
 bug
Bug修复
 ci-bug
CI本身的bug
 ci-optimization
CI优化类任务
 ci-pipeline
CI流水线/workflow相关
 cleanup
代码清理
 devops
运维/部署/CI
 documentation
文档相关
 feature
新功能
 frontend
前端相关
 P0
阻塞性，必须立即处理
 P1
高优先级
 P2
中优先级
 P3
低优先级/后考虑
 question
需要讨论确认
 ready
PR CI通过，可合并
 refactor
代码重构
No Label backend cleanup refactor
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
auto-approve-bot frontend-dev xiaoxia
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1835
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?