Switch to using ECDSA (secp384r1) keys v1 PROPOSED

Mark Dain: 1
 Switch to using ECDSA (secp384r1) keys

 1 files changed, 4 insertions(+), 9 deletions(-)
Export patchset (mbox)
How do I use this?

Copy & paste the following snippet into your terminal to import this patchset into git:

curl -s https://lists.sr.ht/~sircmpwn/gmni-devel/patches/15211/mbox | git am -3
Learn more about email & git
View this thread in the archives

[PATCH] Switch to using ECDSA (secp384r1) keys Export this patch

This patch has been tested with gmni and Castor; both are able to use
certificates generated in this way. Additional testing can be done by
accessing gemini://markdain.net which is now serving an ECDSA cert.

The curve secp384r1 was picked because other curves would fail during
cert generation or would fail in other ways. For instance, secp256k1
would produce a certificate with shorter keys than 384r1, however no
Gemini client would accept it - nor would `openssl s_client'.

Let's Encrypt uses secp384r1 for their new ECDSA root cert, so that
seemed like a safe choice for something widely supported.

 src/tls.c | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/src/tls.c b/src/tls.c
index f7ed344..e3653f2 100644
--- a/src/tls.c
+++ b/src/tls.c
@@ -23,17 +23,12 @@ tls_host_gencert(struct gmnisrv_tls *tlsconf, struct gmnisrv_host *host,
	EVP_PKEY *pkey = EVP_PKEY_new();

	BIGNUM *bn = BN_new();
	BN_set_word(bn, RSA_F4);

	RSA* rsa = RSA_new();
	int r = RSA_generate_key_ex(rsa, 4096, bn, NULL);
	EC_KEY* ec_key = EC_KEY_new_by_curve_name(NID_secp384r1);
	int r = EC_KEY_generate_key(ec_key);
	assert(r == 1);

	EVP_PKEY_assign_RSA(pkey, rsa);
	EVP_PKEY_assign_EC_KEY(pkey, ec_key);

	X509 * x509 = X509_new();

To git@git.sr.ht:~sircmpwn/gmnisrv
   cb2c84b..c538c2a  master -> master