We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Describe the bug In the source documentation: https://github.com/phalcon/cphalcon/blob/v4.0.0/phalcon/Session/Adapter/Redis.zep , it was shown that the options can accept prefix option. However it was hardcoded back into 'sess-reds' in the constructor no matter what was set.
To Reproduce Steps to reproduce the behavior:
$options = [ 'host' => '127.0.0.1', 'port' => 6379, 'index' => '0', 'prefix' => 'custom-prefix-' ]; $session = new Manager(); $serializerFactory = new SerializerFactory(); $factory = new AdapterFactory($serializerFactory); $redis = new Redis($factory, $options); $session ->setAdapter($redis) ->start();
Result of session key in redis:
> KEYS * sess-reds-h4h1fa8oh5epr3vf4upabbknni sess-reds-f1tbr5tjsskrj6vtp4uieleva4
Expected behavior The KEYS in redis session store should be using the custom-prefix-xxxxx instead of default sess-reds
custom-prefix-xxxxx
sess-reds
> KEYS * custom-prefix-h4h1fa8oh5epr3vf4upabbknni custom-prefix-f1tbr5tjsskrj6vtp4uieleva4
Details
Additional context Add any other context about the problem here.
The text was updated successfully, but these errors were encountered:
Fixed in #15185
Sorry, something went wrong.
zsilbi
No branches or pull requests
Describe the bug
In the source documentation: https://github.com/phalcon/cphalcon/blob/v4.0.0/phalcon/Session/Adapter/Redis.zep , it was shown that the options can accept prefix option. However it was hardcoded back into 'sess-reds' in the constructor no matter what was set.
To Reproduce
Steps to reproduce the behavior:
Result of session key in redis:
Expected behavior
The KEYS in redis session store should be using the
custom-prefix-xxxxx
instead of defaultsess-reds
Details
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: