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

[DO NOT MERGE] Build with rustls instead of OpenSSL #896

Closed
wants to merge 2 commits into from

Conversation

charlespierce
Copy link
Contributor

Experimenting how using Rustls affects final binary size

@charlespierce
Copy link
Contributor Author

Looks like ring doesn't like to compile on CentOS 6 since it's so old, but that's the level we've been supporting, so using RusTLS would mean changing our supported environments.

@charlespierce charlespierce deleted the rustls_experiment branch December 10, 2020 19:52
@chriskrycho
Copy link
Contributor

@charlespierce @dherman @thoov @rwjblue thoughts on leaving behind CentOS 6 as one of the things we do with 1.0? Even if that means we cut a 0.10.0 with support for it first, and then 1.0.0 which drops it (and does e.g. this if it's useful).

@charlespierce
Copy link
Contributor Author

@chriskrycho It's not specifically CentOS 6, we were using that to get the CentOS flavor of OpenSSL, however using an older docker image allows us to create a build linked against an older libc version than the oldest GH Actions runner supports (Ubuntu 16.04). That makes Volta work on a wider range of Linux distros out of the box.

I'm hesitant to drop that support, since there's such a variety of distros and support timeframes. For reference, I believe rustup is actually built against CentOS 5, to support an even wider range of possible distros. If, in the future, we decide we want to use RusTLS, we can probably do it in a backwards-compatible way by creating a feature flag and still building the old OpenSSL versions on an older image. It will require making the installer script a little smarter, but that shouldn't be too hard.

@chriskrycho
Copy link
Contributor

Oh, I see. Thank you for the clarification!

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 this pull request may close these issues.

2 participants