Skip to content

Conversation

@xrgzs
Copy link
Member

@xrgzs xrgzs commented Jan 13, 2026

Description / 描述

修复复制链接的生成方式

单文件分享不能包含路径(文件名)

Motivation and Context / 背景

Closes OpenListTeam/OpenList#1968

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).
    我已相应更新了相关仓库(若适用)。

@xrgzs xrgzs requested a review from KirCute January 13, 2026 05:44
@xrgzs xrgzs added the bug Something isn't working label Jan 13, 2026
Copy link
Member

@KirCute KirCute left a comment

Choose a reason for hiding this comment

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

分享分为以下三种情况:

  1. 分享单文件
  2. 分享单文件夹
  3. 分享多文件/文件夹

我测了下,之前1的复制链接有问题,23没问题,现在是1的没问题,23有问题

@xrgzs
Copy link
Member Author

xrgzs commented Jan 13, 2026

我测了下,之前1的复制链接有问题,23没问题,现在是1的没问题,23有问题

我这边2、3的复制链接没测出来问题

@KirCute
Copy link
Member

KirCute commented Jan 13, 2026

我这边2、3的复制链接没测出来问题

我懂了,在右键菜单里和Download页里复制链接复制出来的内容不一样,现在2、3右键菜单里复制的结果不正确

@xrgzs
Copy link
Member Author

xrgzs commented Jan 13, 2026

@KirCute 感觉后端也不太对,单文件分享的 raw_url 最后还多个 /

a4ec13775c17cded44f01d4210051b53

@xrgzs
Copy link
Member Author

xrgzs commented Jan 13, 2026

我认为这个问题应该由后端解决,将文件名添加到 raw_url 后面

@xrgzs xrgzs closed this Jan 13, 2026
@KirCute
Copy link
Member

KirCute commented Jan 13, 2026

单文件分享的下载链接结尾没有文件名,多了一个/确实是问题。

这不应该只是一个后端问题,因为右键菜单里的复制链接和下载界面里的复制链接复制的内容不同,这说明至少有一个链接是前端拼接出来的,所以首先要确认前端拼接的链接是否正确

@KirCute
Copy link
Member

KirCute commented Jan 13, 2026

如果只有单文件分享的复制链接存在问题,我觉得可以考虑让Download页的复制链接直接返回rawURL

@xrgzs
Copy link
Member Author

xrgzs commented Jan 13, 2026

如果只有单文件分享的复制链接存在问题,我觉得可以考虑让Download页的复制链接直接返回rawURL

我也认为 Preview (包括Download)都不应该使用 useLink 这个 hook

但这样会让复制转链的用户不方便

还有要确定一下其它 useLink 的 Preview 有没有问题

@KirCute
Copy link
Member

KirCute commented Jan 16, 2026

这个现在打算怎么办

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] 分享链接访问下的复制链接不对

2 participants