Archived
fix: 统一 core/auth_system 的数据目录解析,消除脑裂隐患
为什么:`core/db.py` 认 SITE_DATA_DIR、`auth_system/db.py` 写死 Path('data'),
而两处指向的是**同一个** data/site.db。web/main.py 的文档字符串还宣称
「数据目录可由 SITE_DATA_DIR 覆盖」——只有 core 侧做到了。
后果:只要运维真的设置 SITE_DATA_DIR,站点库(用户/账本/卡密)与
core 侧读的设置/公告就会落到两个不同文件,且**不会报错**——
表现为「设置改了不生效」「公告消失」这类极难排查的诡异现象。
全仓 grep 确认该变量当前只出现在 core/db.py 与 web/main.py 两处,
没有任何测试或部署在用,所以是纯隐性隐患。
顺带修一个实测到的坑:`os.environ.get('SITE_DATA_DIR', 'data')` 的默认值
只在变量**缺失**时生效,`SITE_DATA_DIR=`(空串,compose/K8s 里很常见)
会让 Path('') 退化成当前工作目录,数据库静默落到 CWD。改用 `or 'data'`。
验证:四种取值(未设置 / 空串 / 绝对路径 / 相对路径)下
core.DATA_DIR == auth_system.DATA_DIR 且 DB_PATH 相同,全部一致。
This commit is contained in:
1 parent
961ee87511
commit
bd3a4ac70b
2 files changed
+10
-3
No files matched your search
+4
-1
@@ -20,7 +20,10 @@ from .security import (
|
|||||||
)
|
)
|
||||||
|
|
||||||
# ============ 路径与常量 ============
|
# ============ 路径与常量 ============
|
||||||
DATA_DIR = Path('data')
|
# 必须与 core.db.DATA_DIR 用**同一条**表达式:两处指向同一个 data/site.db。
|
||||||
|
# 此前这里写死 Path('data') 而 core 侧认 SITE_DATA_DIR——一旦运维设置该变量,
|
||||||
|
# 站点库与设置/公告库会落到两个不同文件(脑裂),且不会报错。
|
||||||
|
DATA_DIR = Path(os.environ.get('SITE_DATA_DIR') or 'data')
|
||||||
|
|
||||||
DB_PATH = DATA_DIR / 'site.db'
|
DB_PATH = DATA_DIR / 'site.db'
|
||||||
|
|
||||||
|
|||||||
+6
-2
@@ -21,8 +21,12 @@ from typing import Any
|
|||||||
|
|
||||||
import aiosqlite
|
import aiosqlite
|
||||||
|
|
||||||
# 站点数据目录(可经环境变量覆盖,便于测试隔离)
|
# 站点数据目录(可经环境变量覆盖,便于测试隔离)。
|
||||||
DATA_DIR = Path(os.environ.get('SITE_DATA_DIR', 'data'))
|
# 用 `or 'data'` 而不是 get 的默认值:get 的默认只在变量**缺失**时生效,
|
||||||
|
# 而 `SITE_DATA_DIR=`(空串)会让 Path('') 退化成当前工作目录,
|
||||||
|
# 数据库静默落到 CWD 而不是 data/。
|
||||||
|
# 注意:auth_system.db.DATA_DIR 必须与此处保持同一条表达式(两者指向同一个库)
|
||||||
|
DATA_DIR = Path(os.environ.get('SITE_DATA_DIR') or 'data')
|
||||||
DB_PATH = DATA_DIR / 'site.db'
|
DB_PATH = DATA_DIR / 'site.db'
|
||||||
|
|
||||||
# 锁按事件循环隔离:asyncio.Lock 绑定创建它的 loop,跨 loop 复用会抛
|
# 锁按事件循环隔离:asyncio.Lock 绑定创建它的 loop,跨 loop 复用会抛
|
||||||
|
|||||||
Reference in new issue
Block a user