Skip to content

Conversation

jellevdd
Copy link

SSL support can be added by including a boost::asio::ssl::context to the server options. If no SSL context is set, the server will behave in the original way (http server).
This approach should be backwards compatible and it should be fairly easy to add SSL support to already existing http servers.

Add basic SSL support to async_server. Use stream_handler class to make
sure we don't copy code.
@jellevdd jellevdd closed this May 15, 2014
@jellevdd jellevdd reopened this May 15, 2014
@patlecat
Copy link

Still using the OpenSSLlib?

@jellevdd
Copy link
Author

It uses boost::asio ssl support, which uses OpenSSL.

@jellevdd
Copy link
Author

@deanberris Could you take a look at the pull request? Thanks!

@deanberris
Copy link
Member

Thanks @jellevdd -- this mostly LGTM.

If you can also update the documentation to show how to set up the server so that it does to SSL on the server as well in another pull request that would be great. :)

deanberris added a commit that referenced this pull request May 19, 2014
@deanberris deanberris merged commit e46a746 into cpp-netlib:0.11-devel May 19, 2014
@jellevdd
Copy link
Author

@deanberris Initial documentation can be found at cpp-netlib/cpp-netlib.github.com#7

@jellevdd jellevdd deleted the 0.11-devel-integration branch May 20, 2014 14:31
@jellevdd jellevdd restored the 0.11-devel-integration branch May 21, 2014 07:29
@patlecat
Copy link

Can the LibreSSL lib be linked easily instead of OpenSSL? Does anyone know?
http://www.libressl.org/

@jellevdd
Copy link
Author

I think it should be possible, they said they aim to be a drop in replacement (not changing any of the exposed interfaces). I'm not sure though, you could ask on the asio mailing list if you want a definite answer (only reason openssl is needed is because boost::asio uses it for ssl).

leecoder pushed a commit to leecoder/cpp-netlib that referenced this pull request Apr 14, 2015
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.

3 participants