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

Create EcKey without intermediate Pkey in gen_vapid_key() #59

Merged
merged 1 commit into from
Dec 13, 2024

Conversation

Kaleb-Reid
Copy link
Contributor

@Kaleb-Reid Kaleb-Reid commented Dec 6, 2024

This fixes building with LibreSSL as PKey::ec_gen() is not supported by it, whereas EcKey::generate() is.
Fixes #58.

@pulsation
Copy link

Hello @Kaleb-Reid , I had the same problem today and can confirm that thanks to your patch, mollysocket compiles and runs like a charm on OpenBSD 7.6 .
It may also resolve #45 .
Thanks a lot!

@Kaleb-Reid Kaleb-Reid changed the title Use different class for private key in gen_vapid_key() Directly create EcKey without intermediate Pkey Dec 7, 2024
@Kaleb-Reid Kaleb-Reid changed the title Directly create EcKey without intermediate Pkey Directly create EcKey without intermediate Pkey in gen_vapid_key() Dec 7, 2024
@Kaleb-Reid Kaleb-Reid changed the title Directly create EcKey without intermediate Pkey in gen_vapid_key() Create EcKey without intermediate Pkey in gen_vapid_key() Dec 7, 2024
@p1gp1g
Copy link
Member

p1gp1g commented Dec 13, 2024

Thanks :)

@p1gp1g p1gp1g merged commit 98b4918 into mollyim:main Dec 13, 2024
@p1gp1g
Copy link
Member

p1gp1g commented Dec 13, 2024

I had to change the JWT crate because it can generate broken signatures.

@Kaleb-Reid Can you test the main branch with LibreSSL ?
@pulsation Can you test on OpenBSD ?

@pulsation
Copy link

Tested on OpenBSD, everything ok here. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Fails to compile with LibreSSL
3 participants