Wurst7
Wurst7 copied to clipboard
Fix ProfileKeys
ProfileKeys do not work unless the user's account type is MSA. Using ProfileKeys.create() fixes this issue, as it defaults to ProfileKeys.MISSING if the user account type is not MSA.
Walkthrough
Walkthrough
The Wurst Client, a Minecraft utility mod, has updated its authentication method to align with Microsoft's account system by switching from Mojang to Microsoft Account (MSA) authentication. Additionally, it has refactored the way profile keys are managed by adopting a factory pattern, which now creates profile keys using a static create method for better abstraction and potentially easier maintenance.
Changes
| File Path | Change Summary |
|---|---|
.../altmanager/MicrosoftLoginManager.java |
Changed AccountType from MOJANG to MSA in the login method. |
.../mixin/MinecraftClientMixin.java |
1. Changed wurstProfileKeys field type from ProfileKeysImpl to ProfileKeys. 2. Modified wurstProfileKeys instantiation to use ProfileKeys.create method. |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Tips
Chat with CodeRabbit Bot (@coderabbitai)
- You can directly reply to a review comment made by CodeRabbit. Example:
I pushed a fix in commit <commit_id>.Generate unit-tests for this file.
- You can tag CodeRabbit on specific lines of code or entire files in the PR by tagging
@coderabbitaiin a comment. Examples:@coderabbitai generate unit tests for this file.@coderabbitai modularize this function.
- You can tag
@coderabbitaiin a PR comment and ask questions about the PR and the codebase. Examples:@coderabbitai generate interesting stats about this repository from git and render them as a table.@coderabbitai show all the console.log statements in this repository.@coderabbitai read src/utils.ts and generate unit tests.@coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid.@coderabbitai read the files in the src/scheduler package and generate README in the markdown format.
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks.
CodeRabbit Commands (invoked as PR comments)
@coderabbitai pauseto pause the reviews on a PR.@coderabbitai resumeto resume the paused reviews.@coderabbitai reviewto trigger a review. This is useful when automatic reviews are disabled for the repository.@coderabbitai resolveresolve all the CodeRabbit review comments.@coderabbitai helpto get help.
Additionally, you can add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
CodeRabbit Configration File (.coderabbit.yaml)
- You can programmatically configure CodeRabbit by adding a
.coderabbit.yamlfile to the root of your repository. - The JSON schema for the configuration file is available here.
- If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation:
# yaml-language-server: $schema=https://coderabbit.ai/integrations/coderabbit-overrides.v2.json
CodeRabbit Discord Community
Join our Discord Community to get help, request features, and share feedback.
This pull request has been open for a while with no recent activity. If you're still working on this or waiting for a review, please add a comment or commit within the next 7 days to keep it open. Otherwise, the pull request will be automatically closed to free up time for other tasks.
Pull requests should be closed if:
- They have been superseded by another pull request
- They are out of scope or don't align with the project
- They have become obsolete due to other changes
- They have bugs or conflicts that won't be resolved