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

Transaction Signing #317

Closed
nlordell opened this issue Jan 29, 2020 · 2 comments · Fixed by #325
Closed

Transaction Signing #317

nlordell opened this issue Jan 29, 2020 · 2 comments · Fixed by #325

Comments

@nlordell
Copy link
Contributor

nlordell commented Jan 29, 2020

@tomusdrw I see that the transaction signing was removed because of licensing issues with ethsign. I’m very sorry that I didn’t even think to check the license (as it was my contribution).

Would you be willing to accept a PR that implements this using libsecp256k1 directly? It is under MIT (https://github.com/bitcoin-core/secp256k1/blob/master/COPYING) and the rust bindings CC0 (https://github.com/rust-bitcoin/rust-secp256k1/blob/master/LICENSE).

@tomusdrw
Copy link
Owner

@nlordell Yes, I'm willing to bring back the account support as I've mentioned in the PR.

No need to apologize, I suggested using the libs myself (being an author of both) and completely forgot about mismatching license.

Another avenue might be attempting to relicense ethsign to a more permissive license, but I need to check feasibility of that option.

@nlordell
Copy link
Contributor Author

I looked through the dependencies and it looks like it uses parity-crypto which also is licensed under GPL3, I imagine that would complicate things a bit.

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 a pull request may close this issue.

2 participants