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

fix(139): restore the Account handling, partially reverts #7850 #7900

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

abc1763613206
Copy link
Contributor

@abc1763613206 abc1763613206 commented Jan 30, 2025

本更新回滚了 #7850 中的部分修改
可能相关的 issue: #7784

都说是移动 2024 年年底把所有账户都转到了新个人云接口,但我这里按 F12 却仍然只能找到旧的 getDisk ,而且使用新个人云接口上传的文件在官方网页端和客户端里根本看不到,而且也缺失了很多文件。

无奈只能换回老个人云,结果猛报 failed get objs: failed to list objs: 用户不存在 ,读代码一看:怎么把 getAccount 的逻辑注释了。

@abc1763613206
Copy link
Contributor Author

详细的复现截图参考 https://t.me/alist_chat/1128693

@j2rong4cn
Copy link
Contributor

抱歉,看漏了,

@xhofe xhofe merged commit b9f397d into AlistGo:main Jan 30, 2025
8 of 12 checks passed
@j2rong4cn
Copy link
Contributor

j2rong4cn commented Jan 30, 2025

@abc1763613206 你试试把d.Account = splits[1]下面哪个 d.post 注释掉会不会出现用户不存在,我的是新个人云测试了没问题,之前就是漏掉了 d.Account 可以去掉这个请求

@abc1763613206
Copy link
Contributor Author

abc1763613206 commented Jan 30, 2025

@abc1763613206 你试试把d.Account = splits[1]下面哪个 d.post 注释掉会不会出现用户不存在,我的是新个人云测试了没问题,之前就是漏掉了 d.Account 可以去掉这个请求

@j2rong4cn 这个应该就是校验用的,如果新个人云开F12还有这个请求的话我倾向于留着

@j2rong4cn
Copy link
Contributor

@abc1763613206 你试试把d.Account = splits[1]下面哪个 d.post 注释掉会不会出现用户不存在,我的是新个人云测试了没问题,之前就是漏掉了 d.Account 可以去掉这个请求

@j2rong4cn 这个应该就是校验用的,如果新个人云开F12还有这个请求的话我倾向于留着

你看一下d.refreshToken,检测Authorization的有效期就足够了

@abc1763613206
Copy link
Contributor Author

@abc1763613206 你试试把d.Account = splits[1]下面哪个 d.post 注释掉会不会出现用户不存在,我的是新个人云测试了没问题,之前就是漏掉了 d.Account 可以去掉这个请求

@j2rong4cn 这个应该就是校验用的,如果新个人云开F12还有这个请求的话我倾向于留着

你看一下d.refreshToken,检测Authorization的有效期就足够了

你开个PR删掉吧(

@xrgzs xrgzs mentioned this pull request Feb 3, 2025
4 tasks
long2005a1 added a commit to long2005a1/Long-Cloud that referenced this pull request Feb 14, 2025
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.

None yet

3 participants