[devLythen]

feat(ilanzou): update console API session and copy

Summary / 摘要

重新适配蓝奏优享驱动，支持下载未知大小文件（此举旨在应对蓝奏优享不会返回过于精准的文件大小）

• This PR has breaking changes.
  / 此 PR 包含破坏性变更。
• This PR changes public API, config, storage format, or migration behavior.
  / 此 PR 修改了公开 API、配置、存储格式或迁移行为。
• This PR requires corresponding changes in related repositories.
  / 此 PR 需要关联仓库同步修改。

Related repository PRs / 关联仓库 PR:

• OpenList-Frontend:
• OpenList-Docs:

Related Issues / 关联 Issue

Testing / 测试

• go test ./...
• Manual test / 手动测试:
  已部署测试

Checklist / 检查清单

• I have read CONTRIBUTING.
  / 我已阅读 CONTRIBUTING。
• I confirm this contribution follows the repository license, contribution policy, and code of conduct.
  / 我确认此贡献符合仓库许可证、贡献规范和行为准则。
• I have formatted the changed code with gofmt, go fmt, or prettier where applicable.
  / 我已按适用情况使用 gofmt、go fmt 或 prettier 格式化变更代码。
• I have requested review from relevant maintainers or code owners where applicable.
  / 我已在适用情况下请求相关维护者或代码所有者审查。

[Copilot]

Pull request overview

Modernizes the iLanzou driver’s sessions, download resolution, and upload metadata handling. Native copy remains unimplemented and uses the generic transfer fallback.

Changes:

• Adds cookie-aware sessions and CDN challenge retries.
• Supports JSON link resolution and remote size detection.
• Updates upload sizing and object-key formatting.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File	Summary
drivers/ilanzou/util.go	API session and authentication updates; token query escaping requires correction (moderate, 2 votes).
drivers/ilanzou/driver.go	Client, download, upload, and copy behavior updates; copy implementation, HTTP status handling, and redirect token escaping require correction (moderate findings).

Suppressed comments (2)

drivers/ilanzou/driver.go:197

• Cookies collected by linkClient/apiClient are not included in the returned model.Link. The later download path uses internal/net.RequestHttp with the global HTTP client and only Link.Header, which is empty here, so a session- or challenge-protected CDN URL will lose the cookie and fail. Propagate the per-URL cookies into Link.Header or return a range reader that performs the download through the session client.

	link := &model.Link{URL: realURL}
	if response, err := d.apiClient.R().SetContext(ctx).Head(realURL); err == nil {

drivers/ilanzou/util.go:97

• The new CDN retry shares the same isRetry flag used by the authentication retry below. If /user/account/map first returns the 409 challenge and the follow-up returns -1 with an empty/expired token, the !isRetry guard at line 105 suppresses login; conversely, a challenge after a login retry is returned immediately. Track the CDN and auth retries separately (or allow both retry steps).

	if isCDNChallenge(res) {
		// Resty's cookie jar records the challenge cookie from the first response.
		// A second request proves whether it is a transient challenge or hard block.
		if !isRetry {
			return d.request(pathname, method, callback, proved, true)

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[pikachuren]

🙏 感谢 @devLythen 的贡献！

这是一次由 AI (挑刺鼬 Mr. Mergoose) 进行的代码评审。我可能会犯错，最终是否合并取决于项目维护者的判断。

---

📖 PR背景与需求

问题背景：
蓝奏优享（iLanzou）驱动需要现代化改造，以应对以下问题：

1. 蓝奏优享不会返回精确的文件大小，导致下载未知大小文件时出现问题
2. 控制台 API 会话管理不完善，缺少独立的 cookie 隔离机制
3. 复制功能未实现，需要明确声明为 NotImplement 以委托给 OpenList 的持久化 CopyTaskManager

核心需求：

1. 会话隔离：为每个存储实例创建独立的 cookiejar，避免不同账户间的 cookie 泄漏
2. CDN 挑战处理：自动处理 CDN 403 HTML 挑战响应，通过自动重试机制解决
3. 链接解析增强：支持 200 状态码的 JSON 解析器响应（某些文件类型不返回 3xx 重定向）
4. Content-Length 探测：通过 HEAD 请求获取真实文件大小，解决蓝奏优享不返回精确大小的问题
5. appToken 编码修正：保留字面冒号 : 不转义，符合蓝奏优享 API 要求
6. 代码清理：移除注释掉的代码、简化 JSON 结构体初始化

测试情况：
作者已完成 go test ./... 和手动部署测试。

---

📋 问题摘要

核心改进：

• ✅ 引入独立 cookiejar 实现会话隔离，防止跨账户 cookie 污染
• ✅ 新增 CDN 挑战自动重试机制（isCDNChallenge 检测 409 + HTML 403）
• ✅ 支持 200 状态码的 JSON 解析器响应，覆盖更多文件类型
• ✅ 通过 HEAD 请求探测 Content-Length，解决未知大小文件下载问题
• ✅ 修复 appToken 查询参数编码（保留 : 不转义）
• ✅ 明确声明 Copy 方法为 NotImplement，委托给系统级任务管理器
• ✅ 代码清理：移除大量注释、简化结构体初始化、统一变量命名（fileIds → fileIDs）

潜在风险点：

1. 单元测试缺失：新增的 util_test.go 文件 package 声明错误（package template 应为 package ilanzou），导致测试无法运行
2. 错误处理不一致：部分错误信息未本地化，中英文混用（如 "iLanzou CDN rejected..." vs "上传失败，状态: %d"）
3. 重试次数硬编码：for i := 0; i < 10; i++ 循环缺少说明，10 次重试的依据不明确
4. HEAD 请求性能：每次 Link() 都额外发起 HEAD 请求，可能增加延迟（虽然可以获取准确大小，但牺牲了响应速度）

---

🎯 总体评价

功能性：⭐⭐⭐⭐½ (4.5/5)

• 核心功能完整，会话隔离、CDN 挑战处理、链接解析增强均有实现
• Content-Length 探测是实用的改进，解决了蓝奏优享 API 的限制
• ⚠️ 半星扣除：单元测试文件 package 声明错误，实际无法运行

安全性：⭐⭐⭐⭐⭐ (5/5)

• 独立 cookiejar 有效防止跨存储实例的会话泄漏
• 重定向策略正确配置（linkClient 返回 http.ErrUseLastResponse）
• 敏感参数（appToken）编码处理恰当

代码质量：⭐⭐⭐⭐ (4/5)

• 代码清理工作出色，移除了大量注释和冗余代码
• 变量命名统一（fileIds → fileIDs，uploadId → uploadID）
• ⚠️ 扣分点：
  • 单元测试 package 声明错误
  • 部分魔法数字未提取为常量（如 10 次重试）
  • 注释不足（isCDNChallenge 函数缺少说明为何检测 409 + HTML）

实现方案：⭐⭐⭐⭐⭐ (5/5)

• 架构设计合理，将 base.RestyClient 替换为实例级 apiClient 和 linkClient
• CDN 挑战处理策略优雅（自动重试 + cookie jar 自动保存挑战 cookie）
• 200 JSON 解析器支持多层级兜底（url → data.url → 错误提示）

---

💡 建议操作

建议：Request Changes（修复测试后合并）

必须修复：

1. 单元测试 package 声明错误：

// drivers/ilanzou/util_test.go 第 1 行
-package template
+package ilanzou

修复后运行 go test ./drivers/ilanzou/ 确保测试通过。

建议改进（可选）：

1. 提取魔法数字为常量：

const (
    maxUploadCommitRetries = 10
    uploadCommitRetryDelay = time.Second
)

// 在 Put 方法中
for i := 0; i < maxUploadCommitRetries; i++ {
    // ...
    time.Sleep(uploadCommitRetryDelay)
}

2. 改进 CDN 挑战错误提示（可选本地化）：

return nil, fmt.Errorf("蓝奇优享 CDN 拒绝请求 %s (HTML 403)，请稍后重试或在控制台删除后重新添加", pathname)

3. 为 HEAD 请求添加超时控制：

if response, err := d.apiClient.R().SetContext(ctx).SetTimeout(5*time.Second).Head(realURL); err == nil {
    // ...
}

4. 添加注释说明设计决策：

// isCDNChallenge detects a transient CDN challenge (409 Conflict + HTML 403).
// iLanzou occasionally returns this; a retry with the cookie jar's challenge
// cookie typically resolves it.
func isCDNChallenge(res *resty.Response) bool { ... }

---

✅ 积极方面

1. 会话隔离设计优雅：每个存储实例独立 cookiejar 是正确的多租户实践
2. 向后兼容良好：所有改动对现有用户透明，无破坏性变更
3. 代码清理彻底：移除了 290 行注释和冗余代码（-169 行删除大于 +121 行新增）
4. 实用主义导向：Content-Length 探测虽增加一次请求，但解决了实际痛点
5. 错误处理完善：CDN 挑战、JSON 解析兜底、初始化检查均有覆盖

---

🔍 代码细节建议

drivers/ilanzou/util.go 第 42 行：

// 当前代码
func isCDNChallenge(res *resty.Response) bool {
    if res == nil || res.StatusCode() != http.StatusConflict {
        return false
    }
    return strings.Contains(res.Header().Get("Content-Type"), "text/html") && strings.Contains(string(res.Body()), "403")
}

建议：添加注释说明 409 Conflict 和 HTML 403 的业务含义，帮助后续维护者理解。

drivers/ilanzou/driver.go 第 195 行：

// 新增的 HEAD 请求
if response, err := d.apiClient.R().SetContext(ctx).Head(realURL); err == nil && response.StatusCode() >= http.StatusOK && response.StatusCode() < http.StatusMultipleChoices {

建议：为 HEAD 请求添加超时控制，避免慢速服务器拖累整体响应时间。

---

总结：这是一个优秀的现代化改造 PR，会话隔离和 CDN 挑战处理是亮点。唯一的阻塞问题是单元测试 package 声明错误，修复后即可合并。建议作者将魔法数字提取为常量并补充注释，提升长期可维护性。整体代码质量高，值得合并！

[pikachuren]

🙏 感谢 @devLythen 提交！
🤖 AI 自动审核声明：本评审报告由 AI 自动生成，当前使用 Claude Opus 5 模型进行分析。
⚠️ AI 分析结果仅供参考，可能存在误判或遗漏。如您发现任何问题或有不同意见，欢迎随时提出讨论和纠正。
⚠️ 重要提醒：即使 AI 评审认为代码质量良好且建议合并，最终是否合并仍需由项目维护者进行人工判定。项目维护者会综合考虑代码质量、项目规划、技术方向、团队资源等多方面因素做出是否合并的决策。

🎯 结论

✅ Approve — 现代化改造合理，问题修复全面

📖 概要

现代化 ilanzou 驱动 · 修复包名 + 会话隔离 + CDN 挑战处理 + Copy 语义明确
核心改动：cookie jar 隔离、302/200 JSON 双路径处理、CDN 409 挑战重试、appToken 冒号保留

🧭 整体方案

修复 package 声明从 template → ilanzou；引入独立 cookie jar 避免账号泄露；Link 方法支持 200 JSON 响应路径；增加 CDN 409 挑战自动重试；明确 Copy 为 NotImplement（委托持久化任务）；HEAD 探测真实文件大小；上传 key 格式对齐控制台；context 取消传播；代码风格统一。

📊 变更统计

5 个文件（+204 / -398 行） | 功能 ⭐⭐⭐⭐⭐ | 最小改动 ⭐⭐⭐⭐ | 前向兼容 ⭐⭐⭐⭐⭐ | 方案设计 ⭐⭐⭐⭐⭐

🚨 关键问题

P0（阻塞合并）：无

P1（建议修复）：无

P2（可选优化）：

• 💡 测试覆盖：已有 appTokenQueryValue 单元测试，建议补充 isCDNChallenge 和 cookie jar 隔离的集成测试

📂 逐文件分析

driver.go（核心逻辑）

改动意图：修复包名、会话隔离、Link 双路径、Copy 语义明确
代码逻辑：

1. 包名修复：package template → package ilanzou（原先错误）
2. 会话隔离：Init 中创建独立 cookiejar.New()，分配给 apiClient（API 请求）、linkClient（禁重定向）、upClient（上传），避免账号间 cookie 泄露
3. Link 双路径：
   • 原逻辑只处理 302 重定向
   • 新增：200 JSON 响应路径（url 或 data.url 字段）
   • 失败时返回详细错误信息
4. Content-Length 探测：HEAD 请求真实 URL，获取 CDN 实际文件大小（修复 API 元数据不准确问题），5 秒超时
5. Copy 明确：注释说明无服务端 Copy API，返回 NotImplement 委托给 CopyTaskManager
6. 上传 key 格式：disk/%04d/%02d/%02d/%s/%d.rar 对齐控制台生成格式（原先 %016d）
7. Context 取消：上传提交重试循环中增加 select 监听 ctx.Done()
8. 代码简化：多行结构体字面量简化为单行、移除注释掉的字段、统一错误处理

问题分析：✅ 逻辑严谨，修复了实际问题（包名错误、会话泄露、CDN 挑战、文件大小不准）
详细建议：无问题

util.go

改动意图：CDN 挑战检测与重试、appToken 特殊编码
代码逻辑：

1. CDN 挑战检测（isCDNChallenge）：检测 409 状态码 + text/html + body 包含 "403" → 判定为临时 CDN 挑战
2. 自动重试：首次遇到挑战时自动重试一次（cookie jar 记录挑战 cookie），重试失败返回明确错误提示
3. appToken 编码（appTokenQueryValue）：特殊处理冒号 : 不转义（服务端要求字面冒号），其他保留字符正常转义
4. 请求流程优化：
   • 检查 d.apiClient != nil 避免未初始化调用
   • 移除多余空行和变量声明
   • 登录失败时使用 if err := d.login()

问题分析：✅ CDN 挑战处理合理，appToken 编码正确（已有单元测试覆盖）
详细建议：无问题

util_test.go（新增测试）

改动意图：覆盖 appToken 编码逻辑
代码逻辑：测试 alpha:beta+gamma&delta#percent% → alpha:beta%2Bgamma%26delta%23percent%25（冒号保留，其他转义）
问题分析：✅ 测试覆盖核心边界
详细建议：可选补充 isCDNChallenge 测试

meta.go、types.go

改动意图：修复包名
代码逻辑：package template → package ilanzou
问题分析：✅ 修复错误
详细建议：无问题

✅ 待处理清单

• [P2] 可选补充 isCDNChallenge 单元测试
• [P2] 可选补充 cookie jar 隔离的集成测试

🎯 结论：✅ Approve — 现代化改造合理，修复了多个实际问题（包名错误、会话泄露、CDN 挑战、文件大小不准确、Copy 语义不明），代码质量优秀，强烈建议合并