opened
SidRoberts
username_0
Hello!

* Type: bug fix
* Link to issue: #14260 

**In raising this pull request, I confirm the following:**

- [x] I have read and understood the [Contributing Guidelines](https://github.com/phalcon/cphalcon/blob/master/CONTRIBUTING.md)
- [x] I have checked that another pull request for this purpose does not exist
- [x] I wrote some tests for this PR
- [-] I have updated the relevant CHANGELOG
- [-] I have created a PR for the [documentation](https://github.com/phalcon/docs) about this change

Two important things of note:

* [From OpenSSL 1.1.1, ciphers are lowercased](https://www.php.net/manual/en/function.openssl-get-cipher-methods.php#123319)
* [Memcached::getAllKeys() appears to be broken from 1.4.23](https://www.php.net/manual/en/memcached.getallkeys.php#123793)

I've tried to alter the code to work around these problems but I'd like someone with a fresh set of eyes to double check as I've been looking at this too long. :stuck_out_tongue_winking_eye:
Travis now uses Ubuntu 18.04 for builds
issue
created
sergeyklay
username_1
It's a little bit confusing with the bugs in the getAllKeys() method

- One says it doesn't work with memcached 1.5+: https://github.com/php-memcached-dev/php-memcached/issues/367
- One says it doesn't work with php-memcached before 3.0.1: https://github.com/php-memcached-dev/php-memcached/issues/203#issuecomment-282041918
comment
created
sergeyklay
username_1
@username_0 Could you please change memcached version check to reflect this 
See: https://github.com/php-memcached-dev/php-memcached/pull/315

Make sure you use php-memcached extension >= 3.0.1 and memcached server >= 1.4.24
comment
created
SidRoberts
username_0
I agree that this is very confusing. Currently Travis is using Memcached server 1.5.6 and Memcached extension 3.1.3. In theory, it should work [but it doesn't](https://travis-ci.org/phalcon/cphalcon/jobs/565634267#L797).

I don't like the idea of a test doing version checking. I think the version checks should be in the `getKeys()` method and we should maybe implement code that actually gets all the keys instead of dealing with this mess.

How about keep the version check at >=1.2.24 for now, merge it and then deal with it later?
comment
created
SidRoberts
username_0
Or, as a last ditch effort, what about https://github.com/phalcon/cphalcon/pull/14268/commits/bc0712eeed23fd27f9c07e82640d5e52e24e46b2?
comment
created
sergeyklay
username_1
Thank you
comment