Skip to content

feat(115_open): Add GetObjInfo to accelerate getting link#888

Merged
xrgzs merged 3 commits intoOpenListTeam:mainfrom
sevxn007:feat/115_open
Jul 31, 2025
Merged

feat(115_open): Add GetObjInfo to accelerate getting link#888
xrgzs merged 3 commits intoOpenListTeam:mainfrom
sevxn007:feat/115_open

Conversation

@sevxn007
Copy link
Copy Markdown
Member

115_open 支持通过path获取文件信息 可以避免目录递归获取 加速link的获取
相关pr OpenListTeam/115-sdk-go#2

@xrgzs
Copy link
Copy Markdown
Member

xrgzs commented Jul 29, 2025

@Seven66677731 SDK 的 PR 已合并,可以更新一下 go mod

xrgzs
xrgzs previously approved these changes Jul 29, 2025
@sevxn007 sevxn007 dismissed xrgzs’s stale review July 30, 2025 14:02

The merge-base changed after approval.

@xrgzs xrgzs merged commit 54ae7e6 into OpenListTeam:main Jul 31, 2025
12 checks passed
@sevxn007 sevxn007 deleted the feat/115_open branch August 1, 2025 13:07
Get(ctx context.Context, path string) (model.Obj, error)
}

type GetObjInfo interface {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

为什么要添加新接口?用Getter不行吗?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

为什么要添加新接口?用Getter不行吗?

用 getter 会导致这里也请求一次网盘,这样 list 的操作就相当于请求了两次网盘

dir, err := GetUnwrap(ctx, storage, path)

Comment on lines +134 to +138
func (d *Open115) GetObjInfo(ctx context.Context, path string) (model.Obj, error) {
if err := d.WaitLimit(ctx); err != nil {
return nil, err
}
resp, err := d.client.GetFolderInfoByPath(ctx, path)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

driver.RootID

这里有个问题,115open用的是driver.RootID,当RootFolderID不是0的时候,既挂载的不是根目录,由于使用的是FolderID所以这里的path无法如期工作

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.

3 participants