rust-tuf icon indicating copy to clipboard operation
rust-tuf copied to clipboard

Consider migrating signing to signatory

Open erickt opened this issue 7 years ago • 5 comments

We are writing our own wrapper around boringssl instead of using ring, so it'd be nice to use our library for signing. One way to implementing this without forcing everyone to use the same crypto library would be to use signatory, which abstracts over a number of different libraries.

erickt avatar Aug 22 '18 21:08 erickt

Note that Signatory presently supports Ed25519 and ECDSA with NIST P-256 or secp256k1. I'm open to adding other constructions/curves if need be.

tarcieri avatar Aug 23 '18 14:08 tarcieri

Also if there's interest I can send a PR for this

tarcieri avatar Aug 23 '18 16:08 tarcieri

I think this is reasonable, but we already have a lot of PRs open for restructuring. You can submit one, but there might be a lot of rebasing. Idk if you want to wait until some of this flurry calms down some.

heartsucker avatar Aug 24 '18 11:08 heartsucker

We are writing our own wrapper around boringssl instead of using ring, so it'd be nice to use our library for signing.

Aren't we using ring already? What is "our library" at the moment?

One way to implementing this without forcing everyone to use the same crypto library would be to use signatory, which abstracts over a number of different libraries.

I'm totally lost, is this proposing to use signatory and get rid of anything that we are doing on our own about signing?

I'm totally new to this code base and struggling a bit around keys and signatures, given that it's not the core business of this project I would expect that it's entirely reusing some other solution. So if this is the proposal, I cannot agree more.

cavokz avatar Dec 11 '19 09:12 cavokz

Happy to help with Signatory support. Also note that in addition to YubiHSM2s as hardware-backed key storage, we're working on support for regular YubiKeys here (WIP):

https://github.com/iqlusioninc/yubikey-piv.rs

tarcieri avatar Dec 11 '19 13:12 tarcieri