Skip to content
This repository has been archived by the owner on Feb 28, 2023. It is now read-only.

Prune openssl #87

Open
Xanewok opened this issue Jul 11, 2021 · 3 comments
Open

Prune openssl #87

Xanewok opened this issue Jul 11, 2021 · 3 comments
Labels
enhancement New feature or request

Comments

@Xanewok
Copy link
Contributor

Xanewok commented Jul 11, 2021

There was an attempt to remove openssl in favour of pure Rust implementation at #67, however we still include it via rouille > tiny_http implementation. We track moving off rouille at #39.

The benefit, as I understand it, is to make the build and distribution easier or more portable, however we do build and ship the binary by using the openssl/vendored feature already in Actions and GitLab, so I'm not sure how critical this issue is.

@drahnr @TriplEight is it something that we want to resolve in the nearest future?

@Xanewok Xanewok added the enhancement New feature or request label Jul 11, 2021
@drahnr
Copy link
Contributor

drahnr commented Jul 13, 2021

I agree that this not an immediate priority, #67 will resolve this incidentally.

Generally I consider it an advantage to avoid openssl entirely, mostly due to the fact of requiring a C++ compiler and the general topic of reproducibility. It's a nice to have, eventually.

@TriplEight
Copy link
Contributor

TriplEight commented Jul 14, 2021

I'm in favour of that, I will free us from having openssl as a dependency in many places. But we already wait for so long, definitely not a prio right now.

@Xanewok
Copy link
Contributor Author

Xanewok commented Feb 5, 2022

As I understand, after merging #131 we're still blocked upstream by briansmith/webpki#54 (we can't use rustls without IP support); is that correct @montekki?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants