[新功能] LittleSkin OAuth 登录支持 #5132
Open
+263
−35
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
应该是没有啥是我要做的了（（（ |
如果只是获取 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Close #4941 (关联投票 #4948)
之前的 PR 由于推送时操作不当炸了
现在重开一个，但是可能有点问题.....
目前已知的 Bug
完成登录后会错误的进行刷新登录
但是我不知道为什么，也没法修，只能麻烦大家看看了....
登录完成后可能因为刷新登录失败反复要求进行原始登录此问题已做缓解性修复，应该不会那么频繁了
提交内包含旧版本代码相关部分代码已同步主仓
(手动 Merge)会打印登录信息的代码等我回来再删已删除
以下是计划制作但最终移除的部分
在服务器返回错误时显示 X-Yggdralt-Req-ID
由于 NetRequestOnce 历史遗留的问题，无法通过简单的逻辑微调实现，逻辑部分已在提交前回滚
（用户和皮肤站管理员体验-1）注意事项
申请权限需要（均为必选项）
offline_access openid Yggdrasil.PlayerProfiles.Select Yggdrasil.MinecraftToken.Create
需要在申请邮件里备注用于正式环境