-
-
Notifications
You must be signed in to change notification settings - Fork 252
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
[Feature] 第三方 Yggdrasil OAuth 登录支持 #5132
base: main
Are you sure you want to change the base?
Conversation
应该是没有啥是我要做的了((( |
如果只是获取 Minecraft Token,不需要 角色信息已经包含在 ID Token 中,而且 |
根据最小权限原则,移除 User.Read 移除无用注释
源代码和正文已移除 User.Read 权限要求 |
失误(((
Delete .github/workflows/issue_similarity.yml
[Sync] Update Fork
9 commit behind of Hex-Dragon/PCL2:main
ely.by 也有了类似计划,应该可以合在一起做 |
5557b23
to
e097916
Compare
[Sync] Update Code to make branch up to date with upstream
yushijinhun/authlib-injector#268 Authlib-Injector 那边要搞新协议了 |
能不能求求他别搞了,我真没空跟进…… |
Response by tnqzh123
等 23 号学校放假了看看能不能写掉(bushi |
Close #4941 (关联投票 #4948)
之前的 PR 由于推送时操作不当炸了
现在重开一个,但是可能有点问题.....
目前已知的 Bug
完成登录后会错误的进行刷新登录
但是我不知道为什么,也没法修,只能麻烦大家看看了....
登录完成后可能因为刷新登录失败反复要求进行原始登录此问题已做缓解性修复,应该不会那么频繁了
提交内包含旧版本代码相关部分代码已同步主仓
(手动 Merge)会打印登录信息的代码等我回来再删已删除
以下是计划制作但最终移除的部分
在服务器返回错误时显示 X-Yggdralt-Req-ID
由于 NetRequestOnce 历史遗留的问题,无法通过简单的逻辑微调实现,逻辑部分已在提交前回滚
(用户和皮肤站管理员体验-1)注意事项
申请权限需要(均为必选项)
offline_access openid Yggdrasil.PlayerProfiles.Select Yggdrasil.MinecraftToken.Create
需要在申请邮件里备注用于正式环境