Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add Yggdrasil Login #2611

Merged
merged 7 commits into from
Nov 25, 2023
Merged

Add Yggdrasil Login #2611

merged 7 commits into from
Nov 25, 2023

Conversation

GLDYM
Copy link
Contributor

@GLDYM GLDYM commented Nov 12, 2023 •
edited
Loading

How to use

To use Yggdrasil, u need to change the following in MinecraftClient.ini.

AccountType = "yggdrasil"
AuthServer = { Host = "littleskin.cn", Port = 443 }

When facing a multi-player account, the selectedProfile is None and u need to input the player name to select.

Problems

  • May meeting some errors when trying to join a Velocity Server.
  • I'm poor in C#, some logics may be strange.

Some code comes from https://github.com/Hikari1nVoid/MCC-authlib ,but there are many errors in the original fork.

@GLDYM GLDYM changed the title Add Yggdrasi lLogin Add Yggdrasil Login Nov 12, 2023
@GLDYM GLDYM marked this pull request as draft November 15, 2023 13:07
@GLDYM GLDYM marked this pull request as ready for review November 15, 2023 13:09
@milutinke
Copy link
Member

milutinke commented Nov 20, 2023 •
edited
Loading

is this ready for merge?
Also, thanks for your contribution.

@GLDYM
Copy link
Contributor Author

GLDYM commented Nov 24, 2023

is this ready for merge? Also, thanks for your contribution.

Already Done.

@GLDYM GLDYM force-pushed the master branch 2 times, most recently from 2e7d710 to 8dfcf9c Compare November 24, 2023 14:16
@milutinke
Copy link
Member

milutinke commented Nov 25, 2023 •
edited
Loading

is this ready for merge? Also, thanks for your contribution.

Already Done.

Okay, I am merging it then.
Edit: Built and released

@milutinke milutinke merged commit c6da4e2 into MCCTeam:master Nov 25, 2023
@TechClusterHQ TechClusterHQ mentioned this pull request Apr 28, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@GLDYM @milutinke