Haowei Wen

Results 39 comments of Haowei Wen

`prevent-proxy-connections` 选项默认就是关闭的,我不知道为什么你会打开。 Blessing Skin 对 `prevent-proxy-connections` 的实现存在问题,最好不要使用此功能。

邮箱大小写不一致可能确实会造成问题,这应该是 Blessing Skin 的 bug。

> 1.未在文档中规定 [Username ->UUID API](http://wiki.vg/Mojang_API#Username_-.3E_UUID_at_time) 虽然Mojang实现了该API,但authlib中却并没有使用到这个API。并且常见的启动器、服务端,如HMCL、Spigot,也并未使用到该API。 我在GitHub上[搜索了这段API的URL](https://github.com/search?utf8=%E2%9C%93&q=https%3A%2F%2Fapi.mojang.com%2Fusers%2Fprofiles%2Fminecraft%2F&type=Code),发现使用并不是很多。事实上,我到目前为之还没有碰到过使用它的mod或插件。但既然Mojang有这个API,mod或插件就还是有可能去调用它的。 我打算将其写入文档。但在未发现调用这个API的启动器、服务端、mod或插件前,我会赋予它一个较低的优先级。你觉得如何?

> 2.[UUID->Profile API](https://github.com/to2mbn/authlib-injector/wiki/Yggdrasil%E6%9C%8D%E5%8A%A1%E7%AB%AF%E6%8A%80%E6%9C%AF%E8%A7%84%E8%8C%83#%E6%9F%A5%E8%AF%A2%E5%8D%95%E4%B8%AA%E8%A7%92%E8%89%B2)中存在[导致NPE的定义](https://github.com/to2mbn/authlib-injector/wiki/Yggdrasil%E6%9C%8D%E5%8A%A1%E7%AB%AF%E6%8A%80%E6%9C%AF%E8%A7%84%E8%8C%83#%E8%A7%92%E8%89%B2%E4%BF%A1%E6%81%AF%E7%9A%84%E5%BA%8F%E5%88%97%E5%8C%96) 从你的堆栈跟踪可以看出,authlib调用的是`/sessionserver/session/minecraft/profile/:uuid`这个API。其[文档](https://github.com/to2mbn/authlib-injector/wiki/Yggdrasil%E6%9C%8D%E5%8A%A1%E7%AB%AF%E6%8A%80%E6%9C%AF%E8%A7%84%E8%8C%83#%E6%9F%A5%E8%AF%A2%E5%8D%95%E4%B8%AA%E8%A7%92%E8%89%B2)中已有说明: >响应格式: >``` >{ > // ... 角色信息(包含角色属性。若unsigned为true,还需要包含数字签名。格式见 §角色信息的序列化) >} >``` 可能我的描述有歧义,`包含角色属性`指的就是包含properties。

我发现一个使用 [Username -> UUID at time API] 的 Mod: [SkinsRestorer]。 I found that [SkinsRestorer] is using [Username -> UUID at time API]. [Username -> UUID at time API]: https://wiki.vg/Mojang_API#Username_-.3E_UUID_at_time [SkinsRestorer]:...

@Karlatemp 十分抱歉,我撤回了你在 wiki 上 [Change \`\`\`javascript to \`\`\`json5, Add GET https://api.mojang.com/users/profiles/minecraft/\?at=\ GET https://api.mojang.com/user/profiles/\/names](https://github.com/yushijinhun/authlib-injector/wiki/Yggdrasil-%E6%9C%8D%E5%8A%A1%E7%AB%AF%E6%8A%80%E6%9C%AF%E8%A7%84%E8%8C%83/_compare/c748553e668715579666a19e8188a63e04cb1a5c...54174a682534e044304c50985764fba6579c9a91) 这个提交。 对于将 `javascript` 改为 `json5` 我十分感谢,但因为这和另外两项变更合在了一个提交内,因此我只能全部撤回。您新增的两个 API,是属于对规范的较大变动,而不是对错误的修正,因此必须在讨论并达成一致之后再实施,希望您见谅。此 issue 应该与您的问题主题相符,所以希望您可以在这里表达看法、提出建议等等。 这两个 API 确实是 Mojang 提供的,但是因为使用极少,因此规范一开始并没有把它们纳入。 @xfl03 对此提出了问题,但因为没有遇到由此造成的问题,所以 issue...

> Overall, looks good to me. But you really need to add the ability to embed an icon in the meta-information for preached meta info format. If you want to...

> 只能 PNG 格式是启动器的限制吗? @g-plane It means if you use formats other than PNG, the behavior is undefined.

> In general, I would say that this is not our problem :) The point is to mention in the documentation that the `icon` field might contain the base64-encoded contents...