Add support for SSL connections to gearmand (Issue #43) #72
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This addresses issue #43. It relies on a version of libgearman.so that supports the
setSSL()
API, which is not yet part of any gearmand release. Refer to gearman/gearmand#63 for details on that issue. These changes are based on [email protected]'s patch for some old version of PHP. I heavily modified the code to work with PHP 7.x.These changes probably need
#ifdef HAVE_LIBGEARMAN_SETSSL
(or something like that) around them in order to support libgearman.so's that havesetSSL()
and those that do not.I've never actually tested it with a PHP SSL worker. We only use PHP SSL on the client side here....