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

Add support for Ed25519 key generation #8

Closed
wants to merge 1 commit into from

Conversation

AlexiaChen
Copy link
Owner

@AlexiaChen AlexiaChen commented Dec 31, 2024

Fixes #6

Add support for Ed25519 key generation in the MPVSS implementation.

  • src/mpvss.rs

    • Import ed25519_dalek crate for Ed25519 key generation.
    • Add KeyAlgorithm enum to represent key generation algorithms.
    • Update MPVSS struct to include key_algorithm field.
    • Add generate_ed25519_keypair function to generate Ed25519 key pairs.
    • Update generate_private_key and generate_public_key functions to support both original and Ed25519 algorithms.
  • README.md

    • Update "Initialization" section to include Ed25519 key generation.
    • Add a note about Ed25519 key generation in the "In the futures" section.
  • tests/mpvss_tests.rs

    • Add test_ed25519_key_generation function to verify Ed25519 key generation.

For more details, open the Copilot Workspace session.

Fixes #6

Add support for Ed25519 key generation in the MPVSS implementation.

* **src/mpvss.rs**
  - Import `ed25519_dalek` crate for Ed25519 key generation.
  - Add `KeyAlgorithm` enum to represent key generation algorithms.
  - Update `MPVSS` struct to include `key_algorithm` field.
  - Add `generate_ed25519_keypair` function to generate Ed25519 key pairs.
  - Update `generate_private_key` and `generate_public_key` functions to support both original and Ed25519 algorithms.

* **README.md**
  - Update "Initialization" section to include Ed25519 key generation.
  - Add a note about Ed25519 key generation in the "In the futures" section.

* **tests/mpvss_tests.rs**
  - Add `test_ed25519_key_generation` function to verify Ed25519 key generation.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/AlexiaChen/mpvss-rs/issues/6?shareId=XXXX-XXXX-XXXX-XXXX).
@AlexiaChen
Copy link
Owner Author

Just test github copilot workspace to plan impl and create PR, seems it not good as I want..... So closed it

@AlexiaChen AlexiaChen closed this Dec 31, 2024
@AlexiaChen AlexiaChen requested a review from Copilot December 31, 2024 01:04

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

tests/mpvss_tests.rs:79

  • Ensure that the new functionality is covered by tests, including integration with the MPVSS struct.
let (secret_key, public_key) = generate_ed25519_keypair();
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.

Public keys
1 participant