ooo
|
17cb012be7
|
refactor(p1-01c): 整併 utils/security 與 utils/validators 重複實作
CD Pipeline / deploy (push) Successful in 1m6s
發現 utils/validators.py 已存在且完整重複 utils/security.py 的 9 個函數。
不收拾的話會繼續腐爛 — 立刻整併為單一權威來源。
變更：
- utils/security.py 增加 safe_read_sql（取自 validators.py 較完整版本，含 limit + params）
- utils/security.py ALLOWED_TABLES 取兩份聯集（補上 monthly_summary_analysis,
realtime_sales_daily），避免破壞既有呼叫者
- utils/validators.py 改為純 re-export shim（保 from utils.validators import 不破）
- app.py 移除原 safe_read_sql 重複定義（35 行），改 import utils.security
routes/import_routes.py 不變（它 from utils.validators 走得到 re-export，等下輪統一）。
行數變化: app.py 7,187 → 7,151 (-36)
|
2026-04-28 15:48:41 +08:00 |
|
ooo
|
f7a5f8505f
|
refactor(p1-01a): app.py 安全工具抽到 utils/security.py
CD Pipeline / deploy (push) Successful in 1m8s
從 app.py 抽出純驗證邏輯 (~180 行) 到 utils/security.py：
- ALLOWED_TABLES 白名單常數
- validate_table_name / validate_column_names (SQL injection 防護)
- safe_join (路徑遍歷防護)
- ALLOWED_UPLOAD_EXTENSIONS / ALLOWED_MIME_TYPES
- secure_filename_unicode / allowed_file / validate_upload_file (上傳驗證)
app.py 保留 from utils.security import * 維持 backward compat，
讓 tests/test_path_traversal.py、tests/test_sql_security.py、
tests/test_file_upload.py 不需修改即可繼續使用 from app import xxx。
行數變化: app.py 7,386 → 7,206 (-180)
|
2026-04-28 15:42:44 +08:00 |
|