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

feat(net): update openssl to 0.9 and cookies to 0.4 #975

Closed
wants to merge 1 commit into from
Closed

feat(net): update openssl to 0.9 and cookies to 0.4 #975

wants to merge 1 commit into from

Conversation

SergioBenitez
Copy link

@SergioBenitez SergioBenitez commented Dec 17, 2016

As the title suggests. This change came about because I want to use the latest version of cookies, which requires the latest version of openssl. Furthermore, this allows for compilation with ssl on OS X without messing up the local libraries due to improvements in the openssl crate.

This was referenced Dec 17, 2016
@upsuper
Copy link

upsuper commented Dec 25, 2016

I guess the owner may want a patch based on master rather than 0.9.x branch.

@SergioBenitez
Copy link
Author

@seanmonstar ping

@steveklabnik
Copy link

Landing this and cutting a new version would be great 👍

@mgattozzi
Copy link

This is what I need as using OpenSSL 0.7 for Hyper is blocking using other libraries I want to use.

@drusellers
Copy link

Yes please

@dovahcrow
Copy link

dovahcrow commented Jan 3, 2017

Now hyper becomes the main stumbling stone of projects because of openssl issue actually...

@dirk
Copy link

dirk commented Jan 3, 2017

@seanmonstar, @reem: Anything we can do as a community to help this PR along?

@seanmonstar
Copy link
Member

I haven't been ignoring this, but instead talking out the problem in detail with some folks, and I think I have a solution. I've filed a new issue with the long write up at #985

tl;dr: A proposal to release a new version of synchronous hyper that removes openssl from its dependency list.

@seanmonstar
Copy link
Member

The solution proposed in #985 was used, and 0.10 is now published: https://github.com/hyperium/hyper/releases/tag/v0.10.0

Thanks everyone for pushing on this idea, I really feel like this was the best solution and we can all move forward!

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.

8 participants