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

Implement test serverside encryption #113

Merged
merged 6 commits into from
Jan 4, 2025
Merged

Implement test serverside encryption #113

merged 6 commits into from
Jan 4, 2025

Conversation

Lompandi
Copy link

@Lompandi Lompandi commented Jan 2, 2025

Implemented serveral Encryption method, including

  1. Encryption.encrypt
  2. Encryption.decrypt

and added some new method, such as

  1. Encryption.init_encryption (which will be used to initialize the encryption after LoginPacket is received)
  2. Encryption.compute_shared_key_ecc, which will compute the EDCH based shared secret

@RadiatedMonkey RadiatedMonkey added the C-Proto Category: Protocol label Jan 2, 2025
@RadiatedMonkey RadiatedMonkey self-assigned this Jan 2, 2025
@RadiatedMonkey RadiatedMonkey merged commit a8600f2 into bedrock-crustaceans:encryption Jan 4, 2025
0 of 4 checks passed
@OmniacDev
Copy link
Member

Can we just make sure all CI tests pass before merging? Thanks

@RadiatedMonkey
Copy link
Member

Can we just make sure all CI tests pass before merging? Thanks

This is a separate encryption branch we're working on, not dev. That's why I merged it without the tests passing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@RadiatedMonkey RadiatedMonkey

Labels
C-Proto Category: Protocol
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Lompandi @OmniacDev @RadiatedMonkey @theaddonn