-
Notifications
You must be signed in to change notification settings - Fork 586
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
Fails to build on systems with OpenSSL 1.1.1 #394
Comments
There's a compat branch that's working just fine (for me) with openssl v1.1.1 at ishitatsuyuki/rust-openssl, with a PR open against rust-openssl v0.9.x. Try it yourself with a patch entry in
|
The PR has been closed, because apparently "0.9 is no longer supported" means "even if someone else does the work for us". :| |
Gonna start digging into upgrading some stuff on the serenity side to fix this, because as of now you can't build Serenity under Arch Linux, which is a huge pain. Unfortunately there's another dependency that pulls in the old |
That crate is a fork of a websocket crate that fixed several issues at the time of its creation. The current solution until dependencies are worked out is to use the patch mentioned by @Celti. Said patch allows serenity to work as expected on platforms using newer OpenSSL versions. |
@Mishio595 What particular issues? Just swapped it for |
native-tls v0.1.5 does not detect OpenSSL 1.1.1 when building, causing compile errors on systems that have it installed (such as Arch). This patch swaps out the outdated evzht9h3nznqzwl crate with its upstream websocket crate. Fixes serenity-rs#394
@ColonelThirtyTwo websockets-rs/rust-websocket#137 This causes any large payloads to often fail, for example |
Fixed in v0.6.0. Should be closed when it releases. |
How about 0.5.x users? I can't upgrade to 0.6 right now because the API is completely changed and it would take more time than I have in the immediate future (I'm looking at an upgrade later this month though). I just ran |
Since openssl 0.9 doesn't support OpenSSL 1.1.1,so that OpenSSL with old ver is needed |
See sfackler/rust-openssl#994.
Serenity pulls in native-tls 0.1.5, which depends on an old version of openssl that doesn't build on systems with OpenSSL 1.1.1.
The text was updated successfully, but these errors were encountered: