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

Failed to authenticate with the Redisd server #12736

Closed
limingxinleo opened this Issue Mar 23, 2017 · 1 comment

Comments

Projects
None yet
2 participants
@limingxinleo
Contributor

limingxinleo commented Mar 23, 2017

use Phalcon\Cache\Backend\Redis as BackRedis;
$cache = new BackRedis(
                $frontCache,
                [
                    'host' => $config->redis->host,
                    'port' => $config->redis->port,
                    'auth' => $config->redis->auth,
                    'persistent' => $config->redis->persistent,
                    'index' => $config->redis->index,
                    'prefix' => ':cache:',
                    'statsKey' => '_PHCM',
                ]
            );

Details

When I use cache service by redis and my redis auth is null.
Failed to authenticate with the Redisd server

phalcon/cache/backend/redis.zep

		if fetch auth, options["auth"] {
			let success = redis->auth(auth);

			if !success {
				throw new Exception("Failed to authenticate with the Redisd server");
			}
		}

I think, if auth is null ,not �execute redis->auth(auth);

  • Phalcon version: 3.0.4
  • PHP Version: 7.0.15
  • Operating System:
  • Installation type: Compiling from source || installing via package manager
  • Zephir version (if any):
  • Server: Nginx | Apache | Other
  • Other related info (Database, table schema):

Daredzik added a commit to Daredzik/cphalcon that referenced this issue Apr 21, 2017

if auth is null ignore them
If you send options array with auth key and value is a null ignore them.
fixing #12736

@Daredzik Daredzik referenced this issue Apr 21, 2017

Closed

if auth is null ignore them #12807

2 of 3 tasks complete

@sergeyklay sergeyklay added this to the 3.2.0 milestone Jun 5, 2017

@sergeyklay sergeyklay self-assigned this Jun 5, 2017

@sergeyklay sergeyklay referenced this issue Jun 5, 2017

Merged

Fixed Redis auth #12881

3 of 3 tasks complete
@sergeyklay

This comment has been minimized.

Show comment
Hide comment
@sergeyklay

sergeyklay Jun 5, 2017

Member

Fixed in the 3.2.x branch. Feel free to open new issue if the problem appears again. Thank you for contributing.

Member

sergeyklay commented Jun 5, 2017

Fixed in the 3.2.x branch. Feel free to open new issue if the problem appears again. Thank you for contributing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment