-
Notifications
You must be signed in to change notification settings - Fork 14
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
migrate from ecdsa and secp256k1 to coincurve #45
Comments
This was referenced Jul 17, 2024
prusnak
changed the title
migrate from secp256k1 to coincurve
migrate from ecdsa and secp256k1 to coincurve
Jul 17, 2024
I've never been a fan of this library coming with a signature library by default. How about switching to coincurve and making it optional? |
Now I understand it better what you meant. Yes, makes sense! 👍 |
Done in #46 and released via 2.1.0 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
i think we should just migrate to coincurve - it is well maintained (last secp256k1 release was in 2021) - and even CLN uses coincurve and not secp256k1 for their python plugins
it would be great to also drop the ecdsa dependency and use only coincurve
The text was updated successfully, but these errors were encountered: