GravitLauncher / Launcher

Улучшеный Minecraft лаунчер на основе sashok's742 v3
https://gravitlauncher.com
GNU General Public License v3.0
132 stars 105 forks source link

Проверить причины зависания клиента при получении скина для голов и исправить по возможности #559

Closed JoCat closed 2 years ago

JoCat commented 2 years ago

Баг тривиальный. При запросе на получение скина головы зависает игра, причина вероятнее всего кроется в том, что запрос выполняется синхронно блокируя основной поток игры.

Путь запроса: так как бошки привязываются к никнейму, то перед тем как получить скин нужно узнать uuid игрока делается это через POST запрос: https://api.mojang.com/profiles/minecraft (https://wiki.vg/Mojang_API#Usernames_to_UUIDs) после чего, после полученного uuid делается GET запрос на получение скина: https://sessionserver.mojang.com/session/minecraft/profile/<uuid> (https://wiki.vg/Mojang_API#UUID_to_Profile_and_Skin.2FCape) Похоже на то, что проблема появляется как раз на этапе первого запроса, ибо у меня при разработке лаунчера была аналогичная ситуация)) Судя по всему в коде лаунчера есть этот запрос, но не факт что он работает корректно, либо проблема кроется в серверах. В общем нужно смотреть.


К слову этот баг трижды упоминался в гилде, но похоже как обычно всем было лень заводить ишью, поэтому придётся мне)) image image image image image image image

dima-dencep commented 2 years ago

Сагласен

microwin7 commented 2 years ago

Если что, добавлю, вчера в саппорте речь про ЛолиСервер шла, но возможно это и в других ядрах сделано не верно

microwin7 commented 2 years ago

Игрок должен видеть свою голову со скином, но к примеру на папере или спиготе, когда тестил, около года назад, была такая проблема, что свою видно, а если зайти с другого акка, то будет не моя голова, а того человека, что смотрит на эту голову

dima-dencep commented 2 years ago

Если что, добавлю, вчера в саппорте речь про ЛолиСервер шла, но возможно это и в других ядрах сделано не верно

Я через эсентиалс выдавал голову, выдалась Алекс и лагало оч, без офлайн мода бошки некоторые со скином гиваются, тот же нотч

dima-dencep commented 2 years ago

Головы конечно не нужная дичь но все же

gravit0 commented 2 years ago

Проблема найдена и будет исправлена в ближайшем обновлении. Метод получения uuid из username тут не причем - проблема была в специфическом методе установки texture properties, доставшейся нам в наследство. Мне эта реализация сразу не понравилась, но я решила - "работает - не трогай" Я подготовила сборки 1.7.10, 1.12.2 и 1.16.5, но что бы всё протестировать требуется время

gravit0 commented 2 years ago

Клиенты обновлены