Skip to content

fix(driver/quark_uc): getting object failed after makedir#2050

Closed
mkitsdts wants to merge 3 commits intoOpenListTeam:mainfrom
mkitsdts:quark_makedir
Closed

fix(driver/quark_uc): getting object failed after makedir#2050
mkitsdts wants to merge 3 commits intoOpenListTeam:mainfrom
mkitsdts:quark_makedir

Conversation

@mkitsdts
Copy link
Contributor

@mkitsdts mkitsdts commented Jan 30, 2026

Description / 描述

通过循环强制获取夸克网盘创建文件夹后的fileid,避免后续因夸克网盘缓存与数据库的不一致而带来的错误。

Motivation and Context / 背景

Relates to #2002

How Has This Been Tested? / 测试

拷贝文件到夸克网盘

Checklist / 检查清单

  • I have read the CONTRIBUTING document.
    我已阅读 CONTRIBUTING 文档。
  • I have formatted my code with go fmt or prettier.
    我已使用 go fmtprettier 格式化提交的代码。
  • I have added appropriate labels to this PR (or mentioned needed labels in the description if lacking permissions).
    我已为此 PR 添加了适当的标签(如无权限或需要的标签不存在,请在描述中说明,管理员将后续处理)。
  • I have requested review from relevant code authors using the "Request review" feature when applicable.
    我已在适当情况下使用"Request review"功能请求相关代码作者进行审查。
  • I have updated the repository accordingly (If it’s needed).
    我已相应更新了相关仓库(若适用)。

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR aims to make Quark/UC directory creation more robust by actively re-fetching the created folder’s file ID after MakeDir, reducing errors caused by eventual consistency between Quark’s cache and OpenList’s database.

Changes:

  • Changed QuarkOrUC.MakeDir to return the created/existing directory object (model.Obj) instead of only error.
  • After issuing the create-folder request, repeatedly lists the parent directory and searches for the new folder by name with backoff + context cancellation support.
  • Special-cases a known Quark/UC conflict message to continue lookup rather than failing immediately.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mkitsdts mkitsdts closed this Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Module: Driver Driver-Related Issue/PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants