Skip to content

fix(teldrive): use a valid root path for listings - #3108

Merged
jyxjjj merged 1 commit into
OpenListTeam:mainfrom
spark-328:codex/fix-3107-teldrive-root
Sep 23, 2026
Merged

jyxjjj merged 1 commit into
OpenListTeam:mainfrom
spark-328:codex/fix-3107-teldrive-root

Conversation

@spark-328

Copy link
Copy Markdown
Contributor

Fixes #3107.

What changed

  • Resolve an empty root object path to the configured Teldrive root, then to /.
  • Use that resolved path consistently for initial and paginated listing requests.
  • Build returned object paths from the same path, so root listing results keep the virtual root prefix.

Validation

  • go test ./drivers/teldrive (passes)
  • Added regression coverage that an empty root object with root_folder_path=/configured-root sends path=/configured-root and returns /configured-root/child.

@pikachuren pikachuren left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

🎯 结论

✅ Approve — 代码质量良好,建议合并

📖 概要

fix(teldrive): use a valid root path for listings

📊 评审结果

改动合理,无重大问题发现。代码逻辑清晰,符合项目规范。

🎯 结论:✅ Approve — 建议合并

@jyxjjj
jyxjjj merged commit cadbf87 into OpenListTeam:main Sep 23, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Teldrive driver: storage root lists all files flattened (empty path param sent to backend)

3 participants