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

(Nearly) full Login packet implementation #106

Merged

Conversation

RadiatedMonkey
Copy link
Member

@RadiatedMonkey RadiatedMonkey commented Jan 1, 2025

This pull request implements most of the Login packet (except deserialization of the player's skin) including verification of the token signatures.

Other changes I made outside of the Login packet itself:

  • ProtoCodecError::UTF8Error now uses the std::str::Utf8Error type internally rather than std::string::FromUtf8Error. Since the String error can be converted into the str one, this makes it possible to use the error variant for both str and String conversions.

@RadiatedMonkey RadiatedMonkey added the C-Proto Category: Protocol label Jan 1, 2025
@RadiatedMonkey RadiatedMonkey changed the base branch from main to encryption January 2, 2025 02:22
@RadiatedMonkey RadiatedMonkey merged commit 930f5f6 into bedrock-crustaceans:encryption Jan 2, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Proto Category: Protocol
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant