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.

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

Merge develop -> unmojang, link to wiki page with common authlib-injector servers #111

Merged
merged 7 commits into from
Nov 15, 2023

Conversation

evan-goode
Copy link
Contributor

Adds a link on the "Add authlib-injector" dialog to a wiki page (not yet created) with a table of common API servers. The dialog now reads

Please enter your username (sometimes an email address), password, and the URL of your API server.
See this page on the PollyMC wiki for a list of common API servers.

Caution! Your username and password will be sent to the authentication server you specify!

Here is my draft of the wiki page: https://gist.github.com/evan-goode/6f4aafcbb26b0b4223fd650c9fd34675 (GitHub wikis are not in the source tree, so I don't think I can pull request it). I've marked this PR as a draft since the link in the code needs to be updated after the wiki page is created.

This PR also merges recent changes from the develop branch into the unmojang branch.

evan-goode and others added 5 commits September 5, 2023 00:22
Resolves fn2006#85

Prism Launcher has refactored a lot of their Nix recently. This includes
the changes from
PrismLauncher/PrismLauncher#1574, but I'm not
100% sure that's what fixed the issue here, it could have been some
other change along the way.

Signed-off-by: Evan Goode <[email protected]>
Port over recent Nix packaging changes from Prism Launcher
fix Nix wrapper using wrong JAVA_PATHS env var
@evan-goode
Copy link
Contributor Author

I'm also going to tack on this bugfix to this PR: 4957597. And possibly more to come.

@fn2006
Copy link
Owner

fn2006 commented Nov 15, 2023

Cool, added the gist to the wiki at https://github.com/fn2006/PollyMC/wiki/Alternative-Auth-Servers

Fixes a bug where the wrong account could be marked as "in use" when the
game is launched if there are multiple accounts that share the same
profile ID. This can lead to the launcher trying to refresh an account
that's currently in use by the game, leading to "Invalid session" errors
in-game.

Signed-off-by: Evan Goode <[email protected]>
@evan-goode evan-goode marked this pull request as ready for review November 15, 2023 17:21
@evan-goode
Copy link
Contributor Author

Ok, fixed the link. Should be ready now.

@fn2006 fn2006 merged commit c5dad68 into fn2006:unmojang Nov 15, 2023
18 of 21 checks passed
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.

3 participants