Skip to content

feat(drivers/139): merge multiple share roots - #2783

Merged
jyxjjj merged 5 commits into
OpenListTeam:mainfrom
syscc:feat/139-share-merge
Jul 23, 2026
Merged

feat(drivers/139): merge multiple share roots#2783
jyxjjj merged 5 commits into
OpenListTeam:mainfrom
syscc:feat/139-share-merge

Conversation

@syscc

@syscc syscc commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Summary / 摘要

  • Support multiple 139 share link ids in share mode.

  • Support password form link_id#password.

  • Merge multiple share roots into one root listing instead of showing raw share ids.

  • Keep preview and download behavior as 302 redirects.

  • Add regression tests for multi-share parsing and reference encoding.

  • 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:

Related Issues / 关联 Issue

N/A

Testing / 测试

  • GOTOOLCHAIN=local go test ./drivers/139
  • Manual test / 手动测试: verified merged share root listing and direct 302 preview/download behavior in source-mode local run

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.
    / 我已按适用情况使用 gofmtgo fmtprettier 格式化变更代码。
  • I have requested review from relevant maintainers or code owners where applicable.
    / 我已在适用情况下请求相关维护者或代码所有者审查。

AI Disclosure / AI 使用声明

  • This PR includes AI-assisted content.
    / 此 PR 包含 AI 辅助内容。

Tools used / 使用工具:

  • Codex

Usage scope / 使用范围:

  • Code generation / 代码生成

  • Refactoring / 重构

  • Tests / 测试

  • Review assistance / 审查辅助

  • I have reviewed and validated all AI-assisted content included in this PR.
    / 我已审核并验证此 PR 中的所有 AI 辅助内容。

  • I can reproduce all AI-assisted content included in this PR without any AI tools.
    / 我可以在没有任何 AI 工具的情况下重现此 PR 中包含的所有 AI 辅助内容。

@syscc
syscc marked this pull request as ready for review July 12, 2026 07:32
@syscc syscc closed this Jul 12, 2026
@syscc syscc reopened this Jul 12, 2026
@syscc
syscc marked this pull request as draft July 12, 2026 12:37
@syscc
syscc marked this pull request as ready for review July 12, 2026 12:45
@syscc

syscc commented Jul 12, 2026

Copy link
Copy Markdown
Contributor Author

Updated the PR with a no-account fallback for 139 share links: when original download requires account authentication but the share API provides a preview URL, the driver now returns the preview URL so video playback can still work without cookies. Original file download still requires account authentication.

Comment thread drivers/139/driver.go Outdated
Comment thread drivers/139/share_test.go
@xrgzs
xrgzs force-pushed the feat/139-share-merge branch 2 times, most recently from ee94536 to efe05cb Compare July 22, 2026 08:22
@xrgzs
xrgzs force-pushed the feat/139-share-merge branch from efe05cb to 92ec8ea Compare July 22, 2026 08:43
xrgzs added a commit to xrgzs/OpenList-Docs that referenced this pull request Jul 22, 2026
Signed-off-by: MadDogOwner <[email protected]>
@xrgzs
xrgzs requested a review from Suyunmeng July 22, 2026 09:18
@jyxjjj
jyxjjj merged commit 0a9e71a into OpenListTeam:main Jul 23, 2026
12 checks passed
jyxjjj pushed a commit to OpenListTeam/OpenList-Docs that referenced this pull request Jul 23, 2026
* docs(drivers/139): add option to use old stream upload method for family and group

Signed-off-by: MadDogOwner <[email protected]>

* refactor: use chinese for options

Signed-off-by: MadDogOwner <[email protected]>

* Add OpenListTeam/OpenList#2783

Signed-off-by: MadDogOwner <[email protected]>

---------

Signed-off-by: MadDogOwner <[email protected]>
@syscc
syscc deleted the feat/139-share-merge branch July 23, 2026 06:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Module: Driver Driver-Related Issue/PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants