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

Fetch CurseForge API key from official files #13

Merged
merged 1 commit into from
Jun 24, 2024

Conversation

evan-goode
Copy link
Member

Instead of fetching the CurseForge API key from PolyMC's server, fetch it from the files of the CurseForge app, mirrored by archive.org. We can do this without downloading the entire 100MiB+ application by range-requesting the specific DEFLATE block where the key is and decompressing just that block.

@evan-goode evan-goode force-pushed the evan-goode/curseforge branch from 51caceb to e8c39f8 Compare June 24, 2024 02:20
Instead of fetching the CurseForge API key from PolyMC's server, fetch
it from the files of the CurseForge app, mirrored by archive.org. We can
do this without downloading the entire 100MiB+ application by
range-requesting the specific DEFLATE block where the key is and
decompressing just that block.
@evan-goode evan-goode force-pushed the evan-goode/curseforge branch from e8c39f8 to 810ab86 Compare June 24, 2024 02:35
@evan-goode evan-goode merged commit 5e9b3c2 into develop Jun 24, 2024
39 checks passed
@evan-goode evan-goode deleted the evan-goode/curseforge branch June 24, 2024 03:02
@lemonzest79
Copy link

When running from GUI no contents appear in the Curseforge instance creation menu, when ran from a CLI getting access denied from Curseforge with this key, the older key pulled from PolyMC works fine still

@evan-goode
Copy link
Member Author

Well that was difficult, but I think it's fixed now. It's now downloading the same key as served by PolyMC.

@lemonzest79
Copy link

Works for me, tried importing a few curseforge modpack they all worked, removed the key, it revoked access, added it back with the menu button and it regained access so seems working

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.

2 participants