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.

Sign up for GitHub

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

Jump to bottom

PHP Redis extension support TLS connections since 5.3.0 #140

Closed
akunzai opened this issue Oct 12, 2020 · 0 comments · Fixed by #142
Closed

PHP Redis extension support TLS connections since 5.3.0 #140

akunzai opened this issue Oct 12, 2020 · 0 comments · Fixed by #142

Comments

@akunzai
Copy link

akunzai commented Oct 12, 2020

Maybe we can detect the version of Redis extension and use it for batter performance?

I will try it and submit a PR later.
@akunzai akunzai mentioned this issue Oct 12, 2020
Merged
colinmollenhour pushed a commit that referenced this issue Oct 12, 2020
@akunzai
Fixes #140: Update test suites (#142)
e217dc3 
* Update Dockerfile

- Removed PHP 5.5 & Added PHP 7.4
- PHP Redis Extension 4.3.0 is the latest version to support PHP 5
- Update Redis to 6.0.8 and enable TLS support for PHP 7

* Update .travis.yml

- Update php-redis to 5.3.0
- Update redis-server to 6.0.8

* Fixes test failures

- Use [assertIsArray](sebastianbergmann/phpunit#3368) method since PHPUnit v7.5
- Fixes AUTH error message changed since Redis v6
- `getMock()` is deprecated since PHPUnit v5.4

* Update Client.php

- PHP Redis extension support TLS since [5.3.0](https://pecl.php.net/package/redis/5.3.0)
- Fixes AUTH may not cleanup last error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes #140: Update test suites
1 participant
@akunzai