Skip to content

Fix Microsoft authentication#2986

Merged
Glavo merged 3 commits intoHMCL-dev:mainfrom
zly2006:main
Apr 14, 2024
Merged

Fix Microsoft authentication#2986
Glavo merged 3 commits intoHMCL-dev:mainfrom
zly2006:main

Conversation

@zly2006
Copy link
Copy Markdown
Contributor

@zly2006 zly2006 commented Apr 13, 2024

fix: #2979

@zly2006
Copy link
Copy Markdown
Contributor Author

zly2006 commented Apr 13, 2024

@burningtnt

@burningtnt
Copy link
Copy Markdown
Member

emmm,我看这个请求的返回数据没有被使用,真的是必要的吗?

如果不确定呢,我可以给你合并到 PR Collection 里,推给遇到相同问题的人做测试。

无论如何,感谢贡献~

@zly2006
Copy link
Copy Markdown
Contributor Author

zly2006 commented Apr 13, 2024

emmm,我看这个请求的返回数据没有被使用,真的是必要的吗?

  1. 我觉得wiki.vg不会凭空写这一点,所以这应该是官启的流程,而如你所知官启是能启动的,所以这很可能是原因
  2. 我加上这一行代码立刻成功启动

如果不确定呢,我可以给你合并到 PR Collection 里,推给遇到相同问题的人做测试。

嗯,我觉得应该有用。

@burningtnt
Copy link
Copy Markdown
Member

好的,我给你合并过去,观察一段时间

@zkitefly
Copy link
Copy Markdown
Member

如果真的解决了相关问题,那麻烦将相关报错提示也改一下吧(谢谢啦)

@burningtnt
Copy link
Copy Markdown
Member

报错提示不用改吧……

@zly2006
Copy link
Copy Markdown
Contributor Author

zly2006 commented Apr 13, 2024

如果真的解决了相关问题,那麻烦将相关报错提示也改一下吧(谢谢啦)

如果原因真的如此,那么再登录失败就只能是没买正版了(或者被墙),还要改什么?

@burningtnt
Copy link
Copy Markdown
Member

已经给你合并到 https://github.com/burningtnt/HMCL PR Collection 里了。之后如果有人遇上这个问题,我们会推送过去让他们测试的。感谢贡献

@zkitefly
Copy link
Copy Markdown
Member

如果真的解决了相关问题,那麻烦将相关报错提示也改一下吧(谢谢啦)

如果原因真的如此,那么再登录失败就只能是没买正版了(或者被墙),还要改什么?

就是改一下 account.methods.microsoft.error.no_character 的提示

@burningtnt
Copy link
Copy Markdown
Member

就是改一下 account.methods.microsoft.error.no_character 的提示

如果这个 PR 真的有作用,那再没角色就是真的没角色,不是假的没角色

@burningtnt
Copy link
Copy Markdown
Member

已推送 PR Collection 版本更新。可前往 https://zkitefly.github.io/HMCL-Snapshot-Update/prs 下载

@zkitefly zkitefly requested review from Glavo and huanghongxun April 13, 2024 10:09
@zly2006
Copy link
Copy Markdown
Contributor Author

zly2006 commented Apr 13, 2024

@burningtnt

This comment was marked as outdated.

@burningtnt
Copy link
Copy Markdown
Member

burningtnt commented Apr 14, 2024

已推送给 HMCL QQ 群内的三位遇上问题的人测试,均解决了。佬太强了!这个问题终于找到原因了

@zly2006
Copy link
Copy Markdown
Contributor Author

zly2006 commented Apr 14, 2024

那3.5.8啥时候发? 👀

@zly2006
Copy link
Copy Markdown
Contributor Author

zly2006 commented Apr 14, 2024

插一句不算题外话的题外话,其实当时是因为发现登录我自己写的网站之后就可以正常登录hmcl,对比了一下发现我的网站有mcstore这个请求而hmcl没有,也算是运气好正好写了这么个东西(欢迎来看看:https://redenmc.com)

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.

[Bug] Cannot authentic with Microsoft account

4 participants