Skip to content

fix(drivers/123): update web api host#2677

Closed
mm-o wants to merge 1 commit into
OpenListTeam:mainfrom
mm-o:fix/123-api-host
Closed

fix(drivers/123): update web api host#2677
mm-o wants to merge 1 commit into
OpenListTeam:mainfrom
mm-o:fix/123-api-host

Conversation

@mm-o

@mm-o mm-o commented Jun 29, 2026

Copy link
Copy Markdown

Summary

Update the 123Pan web API host constants to match the current production web client configuration.

  • Change the regular 123Pan API base from https://www.123pan.com/b/api to https://api.123278.com/b/api.

  • Change the generic API base from https://www.123pan.com/api to https://api.123pan.cn/api.

  • Keep the existing origin / referer request headers as https://www.123pan.com/.

  • This fixes login/test/list flows where https://www.123pan.com/b/api/user/info now returns 200 text/html, causing JSON parsing failures.

  • This PR has breaking changes.

  • This PR changes public API, config, storage format, or migration behavior.

  • This PR requires corresponding changes in related repositories.

Close #2675

Related repository PRs:

  • OpenList-Frontend:
  • OpenList-Docs:

Testing

  • gofmt -w drivers/123/util.go
  • git diff --check
  • go test ./...
  • Manual test:

Manual verification performed outside this repository with a real 123Pan account:

  • https://login.123pan.com/api/user/sign_in returns a valid token.
  • https://www.123pan.com/b/api/user/info returns 200 text/html with that token.
  • https://api.123278.com/b/api/user/info returns JSON with that token.
  • https://api.123278.com/b/api/file/list/new returns JSON with that token.

Targeted test attempted:

  • go test ./drivers/123 was attempted, but dependency downloads from proxy.golang.org failed with repeated EOF errors before compilation.

Checklist

  • I have read 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.
  • I have requested review from relevant maintainers or code owners where applicable.

AI Disclosure

  • This PR includes AI-assisted content.

Tools used:

  • ChatGPT
  • Codex
  • GitHub Copilot
  • Claude
  • Gemini
  • Other:

Usage scope:

  • Code generation

  • Refactoring

  • Documentation

  • Tests

  • Translation

  • Review assistance

  • I have reviewed and validated all AI-assisted content included in this PR.

  • I have ensured that all AI-assisted commits include Co-Authored-By attribution.

  • I can reproduce all AI-assisted content included in this PR without any AI tools.

Co-Authored-By: Codex <codex@openai.com>
@xrgzs xrgzs changed the title fix(123): update web api host fix(drivers/123): update web api host Jun 29, 2026
@xrgzs xrgzs added the Module: Driver Driver-Related Issue/PR label Jun 29, 2026
Comment thread drivers/123/util.go
const (
Api = "https://www.123pan.com/api"
Api = "https://api.123pan.cn/api"
AApi = "https://www.123pan.com/a/api"

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.

@mm-o 这个AApi也是返回静态了,是不是也需要更换?

@jyxjjj

jyxjjj commented Jun 30, 2026

Copy link
Copy Markdown
Member

这是官方域名吗

@PIKACHUIM

Copy link
Copy Markdown
Member

感谢您的贡献,但此PR没有 修改AAPI和BAPI,也没有修复123 Share的问题
因此我们使用#2678 实现了修复
本PR将关闭 再次感谢您的贡献

@PIKACHUIM PIKACHUIM closed this Jun 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Module: Driver Driver-Related Issue/PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] 123 cloud drive empty list - missing platform header

4 participants