Redis sess: fix path for persistent_identifier & compression_threshold #9368
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.
Currently, the paths for both
persistent_identifier
&compression_threshold
are prefixed withparam_
. This is against both the documentation and it does not follow the standard used throughout the configuration. This PR removes the prefix.Docs:
https://github.com/magento/devdocs/blob/develop/guides/v2.0/config-guide/redis/redis-session.md#configure-magento-to-use-redis-for-session-storage
Fixed Issues (if relevant)
Currently,
persistent_identifier
andcompression_threshold
are ignored in the Redis session configuration (the documented options for configuring).Manual testing scenarios
Change
compression_threshold
to 1 (to force compression) and the sessions will not be compressed. Changing the name to beparam_compression_threshold
results in compression.Contribution checklist