Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(123_share): add access token #6357

Merged
merged 1 commit into from
Apr 24, 2024
Merged

feat(123_share): add access token #6357

merged 1 commit into from
Apr 24, 2024

Conversation

xrgzs
Copy link
Contributor

@xrgzs xrgzs commented Apr 16, 2024

  • 合并了 123 driver 的一些变动,现在 123_share driver 能够正常使用了
  • 加上access token,现在能够解析大文件了
  • 分享链接密码改为可选

Copy link

welcome bot commented Apr 16, 2024

Thanks for opening this pull request! Please check out our contributing guidelines.

@xhofe xhofe merged commit 32ddab9 into alist-org:main Apr 24, 2024
3 checks passed
Copy link

welcome bot commented Apr 24, 2024

Congrats on merging your first pull request! We here at behavior bot are proud of you!

@huolanyun
Copy link

failed get objs: failed to list objs: 分享码错误或者分享地址错误
https://www.123pan.com/s/vj8eVv-QQ413.html

QQ截图20240425104738

@xrgzs
Copy link
Contributor Author

xrgzs commented Apr 25, 2024

failed get objs: failed to list objs: 分享码错误或者分享地址错误 https://www.123pan.com/s/vj8eVv-QQ413.html

QQ截图20240425104738

您提供的链接我使用最新的CI版本测试可以正常解析

https://github.com/alist-org/alist/actions/runs/8821744365

image

image

@huolanyun
Copy link

奇怪了,我拉取的是最新合并PR

@xrgzs
Copy link
Contributor Author

xrgzs commented Apr 25, 2024

奇怪了,我拉取的是最新合并PR

看一下分享key有没有空格之类的特殊字符,另外最好使用国内住宅IP挂载

@xrgzs
Copy link
Contributor Author

xrgzs commented Apr 25, 2024

奇怪了,我拉取的是最新合并PR

看一下挂载设置的分享key有没有空格之类的特殊字符,另外最好使用国内住宅IP挂载

@huolanyun
Copy link

QQ截图20240425123721
没有空格,国内机器

@huolanyun
Copy link

出来了,直接替换Alist不行,需要重新生成data

@huolanyun
Copy link

之前的驱动太多了,替换后 不生效,不知道怎么迁移驱动

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants