Archived
snowy-yang
22b263f547
为什么：smtplib 是同步栈，原先靠 asyncio.to_thread 卸载——线程池里躺着的仍是一次 完整的 TCP + TLS 握手加多轮 SMTP 交互，属于「同步网络 IO 藏在 to_thread 里」：既 占着线程，也不产生并发收益。这是本系列最后一条此类遗留（aria2p 那条已在 C2 清零）。 改动： - auth_system/mail.py：_smtp_connect / _smtp_send 两个同步函数合成一个原生异步 _smtp_send，底层换 aiosmtplib.SMTP。TLS 口径与原实现一一对应：465 用 use_tls （隐式 TLS），其余端口用 start_tls（显式 STARTTLS，先握手再 LOGIN）；两条路都传 显式构造的 ssl.create_default_context()，不依赖运行时默认校验口径。finally 里 quit 失败只吞连接关闭错误，sendmail 的真实异常原样上抛——send_verify_code 依赖 它决定「不写库、不烧 24h 额度」。 - auth_system/mail.py：send_email 改直接 await；移除 import asyncio / import smtplib。 - auth_system/__init__.py：撤掉 _smtp_connect 的再导出——它是同步时代的辅助函数， 全仓无消费方，新实现里也不存在对应物。 - web/routers/register.py：更新过时的模块说明（原文仍写「smtplib 阻塞发送已卸载」）。 - pyproject.toml / uv.lock：新增 aiosmtplib>=5.1.3,<6.0。 - tests/test_mail_smtp.py（新增 5 条）：SMTP 路径此前零覆盖，而「端口 → TLS 方式」 的映射属「授权码会不会明文过网」级别的差异，一旦被改反不会有任何地方报警。用假 SMTP 替身记录构造参数与调用序、全程不触网：587 必须 start_tls 且 LOGIN 在 connect 之后、465 必须 use_tls 且不得再叠 STARTTLS、端口缺省回落 587、quit 失败不掩盖结果、 sendmail 失败原样上抛且仍关连接。 验证： - uv run ruff check . → All checks passed! - uv run pyright → 0 errors, 0 warnings, 0 informations （aiosmtplib 自带 py.typed，故对它的调用是有效类型校验；另用 inspect 逐一核对了 5.1.3 的 SMTP / connect / login / sendmail / quit 参数名） - uv run pytest → 397 项，0 失败 / 0 error / 0 skipped（C3 后 392，新增 5） - data/site.db sha256 前后一致（a19f5dbf…6487），无 -wal/-shm 残留
128 lines
5.6 KiB
TOML
128 lines
5.6 KiB
TOML
# -*- coding: utf-8 -*-
|
||
[project]
|
||
name = "yunmusic-assistant"
|
||
version = "3.0.0"
|
||
description = "云村小助手 · 网易云音乐解析/下载服务（FastAPI + asyncio + aiohttp）"
|
||
requires-python = ">=3.12"
|
||
dependencies = [
|
||
"fastapi>=0.115,<1.0",
|
||
"uvicorn[standard]>=0.30,<1.0",
|
||
"aiohttp>=3.10,<4.0",
|
||
"aiosqlite>=0.20,<1.0",
|
||
"aiosmtplib>=5.1.3,<6.0",
|
||
"itsdangerous>=2.2,<3.0",
|
||
"cryptography>=50.0,<51.0",
|
||
"qrcode>=7.0,<8.3",
|
||
"python-multipart>=0.0.9",
|
||
"filetype>=1.2.0",
|
||
"mutagen>=1.48.1",
|
||
]
|
||
|
||
[dependency-groups]
|
||
dev = [
|
||
"pytest>=8.0",
|
||
"pytest-asyncio>=0.24",
|
||
"ruff>=0.6",
|
||
"pyright>=1.1.390",
|
||
"httpx>=0.27",
|
||
"httpx2>=2.13.1",
|
||
]
|
||
|
||
# 应用工程（非库）：不把自身装进环境，避免 uv 扫描顶层模块
|
||
[tool.uv]
|
||
package = false
|
||
|
||
# ============ Ruff：静态检查 + 导入排序 + 格式化 ============
|
||
[tool.ruff]
|
||
target-version = "py312"
|
||
line-length = 120
|
||
extend-exclude = [".venv", ".zcode", "templates", "static", "frontend", "data"]
|
||
|
||
[tool.ruff.lint]
|
||
# E/F 基础错误、I 导入排序、B bugbear、UP 升级、C4 推导式、ASYNC 异步陷阱、S 安全、RUF
|
||
select = ["E", "F", "W", "I", "B", "UP", "C4", "ASYNC", "S", "RUF"]
|
||
ignore = [
|
||
"E501", # 行宽交给 formatter，不强制
|
||
"S101", # 生产代码外的 assert（测试/断言脚本）
|
||
"S104", # 绑定 0.0.0.0（服务端有意为之）
|
||
"S110", # try/except/pass（本项目「失败静默降级」策略）
|
||
"S112", # try/except/continue
|
||
"S324", # hashlib.md5 用于非安全场景（上游签名口径）
|
||
"S603", "S607", # subprocess（打包/运维脚本）
|
||
"UP009", # 保留 `# -*- coding: utf-8 -*-` 文件头（全项目既有约定）
|
||
"RUF001", "RUF002", "RUF003", # 全角字符（中文注释/文案，误报）
|
||
# ---- 以下为「有意保留」，已被逐条论证为协议/设计约束，非待修缺陷 ----
|
||
"RUF012", # 常量类可变默认值：APIConstants/PodcastConstants 是常量表，非实例状态
|
||
"S305", # AES-ECB：网易云 weapi 协议**要求**的加密模式，非可自由选择
|
||
"S311", # randrange：生成上游 requestId（协议随机数），非安全用途
|
||
"ASYNC250", # input()：music_api.qr_client 交互式 CLI 的人类输入，异步化无意义
|
||
"S310", # urllib：仅用于音频 Range 代理的 URL 校验路径
|
||
]
|
||
|
||
[tool.ruff.lint.per-file-ignores]
|
||
"tests/**" = ["S", "ASYNC"]
|
||
".zcode/**" = ["S", "ASYNC", "F"]
|
||
# ASYNC109：timeout 参数是有意 API——直接转发给 aiohttp 的 ClientTimeout，
|
||
# 不是「用 sleep 当超时」的反模式。
|
||
"core/http_client.py" = ["ASYNC109"]
|
||
# ASYNC109：timeout 是**总下载时限**（交给 aria2 轮询循环判定），不是「用 sleep 当超时」。
|
||
# ASYNC240：搬迁后才出现的阻塞文件操作只有 unlink（删残片/控制文件）——目录项删除，
|
||
# 微秒级且不碰网络，卸载到线程的调度开销大于操作本身，与 downloader.py 同口径。
|
||
"music_downloader/_aria2.py" = ["ASYNC109", "ASYNC240"]
|
||
# ASYNC230/240：残留的阻塞文件系统调用都是**本地元数据/短写**——exists()/stat()/
|
||
# os.replace()/往 .part 顺序写分块，微秒级且不碰网络，卸载到线程的调度开销大于操作
|
||
# 本身。真正的重活（mutagen 全量重写标签、zipfile 整段拷贝、shutil.rmtree、同步
|
||
# SQLite 落库）都已各自 to_thread 卸载；aria2 的 RPC 已改为手写原生异步实现
|
||
# （music_downloader/_aria2.py），不再是阻塞调用。
|
||
"music_downloader/downloader.py" = ["ASYNC230", "ASYNC240"]
|
||
"services/podcast.py" = ["ASYNC230", "ASYNC240"]
|
||
"services/batch.py" = ["ASYNC240"]
|
||
# ASYNC110：暂停闸门轮询的是**跨线程**置位的 threading.Event 与任务表状态——置位方
|
||
# 在请求线程/监控线程，不属于本事件循环，故不能用 asyncio.Event 取代（见 _taskcore
|
||
# 模块 docstring 的取消/暂停载体说明）。
|
||
"services/_taskcore.py" = ["ASYNC110"]
|
||
|
||
[tool.ruff.lint.isort]
|
||
known-first-party = ["core", "web", "services", "music_api", "auth_system", "music_downloader"]
|
||
|
||
[tool.ruff.format]
|
||
quote-style = "preserve"
|
||
|
||
# ============ Pyright：类型检查 ============
|
||
[tool.pyright]
|
||
# 全量纳入（Flask 巨石删除后不再有例外清单）
|
||
# 顶层模块与 tests 此前不在名单里——「pyright 0 错误」只覆盖了 core/web/services，
|
||
# 而早期 requests→aiohttp 迁移的遗留恰好全部躲在漏检的顶层模块中，故补齐。
|
||
include = [
|
||
"core",
|
||
"web",
|
||
"services",
|
||
"tests",
|
||
"music_api",
|
||
"auth_system",
|
||
"music_downloader",
|
||
"_mkpkg.py",
|
||
]
|
||
exclude = [".venv", ".zcode", "data", "downloads", "templates", "static", "frontend"]
|
||
venvPath = "."
|
||
venv = ".venv"
|
||
pythonVersion = "3.12"
|
||
typeCheckingMode = "standard"
|
||
reportMissingImports = true
|
||
# standard 比 basic 多出「变量可能未绑定 / 实参类型不符 / 无意义重定义 / 未使用符号」
|
||
# 等**可执行缺陷**（basic 全部漏报，实测 standard 多报 7 处真实缺陷），
|
||
# 故作为门禁级别；strict 的 reportUnknown* 需要给全部第三方库补存根，暂不启用。
|
||
# 以下三项在 standard 下已由类型系统覆盖，降为 warning 以避免第三方存根不全的噪声
|
||
reportOptionalMemberAccess = "warning"
|
||
reportAttributeAccessIssue = "warning"
|
||
|
||
# ============ Pytest ============
|
||
[tool.pytest.ini_options]
|
||
asyncio_mode = "auto"
|
||
testpaths = ["tests"]
|
||
addopts = "-q"
|
||
# 本工程不装进环境（package = false），且 tests/ 无 __init__.py，
|
||
# 故必须显式把项目根加入 import 路径——否则裸跑 `pytest` 时
|
||
# `import core` 必然 ModuleNotFoundError（只有 `python -m pytest` 才碰巧能用）。
|
||
pythonpath = ["."]
|