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

Edwards curves are not compatible with ECDH #348

Open
Sarkoxed opened this issue Oct 4, 2024 · 1 comment
Open

Edwards curves are not compatible with ECDH #348

Sarkoxed opened this issue Oct 4, 2024 · 1 comment
Labels
feature functionality to be implemented help wanted
Milestone
someday/future

Comments

@Sarkoxed
Copy link

Sarkoxed commented Oct 4, 2024 •
edited
Loading

After trying to initialize ECDH with Ed25519 and compute a shared secret it returns the following error:

AttributeError: 'PrivateKey' object has no attribute 'secret_multiplier'

I believe it's called __s in eddsa.PrivateKey class.
@tomato42
Copy link
Member

tomato42 commented Oct 4, 2024

yes, because that's Ed25519, ECDH operation over Curve25519 is X25519, that's not implemented.

it's documented in the README.md:

Key generation, siging and verifying is also supported for Ed25519 and Ed448 curves.

@tomato42 tomato42 added help wanted feature functionality to be implemented labels Oct 4, 2024
@tomato42 tomato42 added this to the someday/future milestone Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature functionality to be implemented help wanted
Projects
None yet
Milestone
someday/future
Development

No branches or pull requests
2 participants
@tomato42 @Sarkoxed