Sessions: SessionConfig does not allow setting non-directory save path #2932

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

mwillbanks commented Nov 10, 2012

The StandardConfig defines that the only way to specify a savePath is to utilize a directory. The SessionConfig then requires the savePath to be a directory since it inherits from the StandardConfig.

This pull request is to check in SessionConfig to see if the save_handler option is "files" and issue the set to the parent otherwise to skip checking.

This means it will allow the following use cases:
files => /path/to/directory
sqlite => /path/to/my.db
memcached => sess1:11211, sess2:11211
redis => tcp://0.0.0.0:6379

@weierophinney weierophinney added a commit that referenced this pull request Nov 16, 2012

@weierophinney weierophinney [#2932] different save_handler in tests
- "sqlite" save handler is not installed by default; changed test to use
  "user" save handler instead
9bd84cd

@weierophinney weierophinney added a commit that referenced this pull request Nov 16, 2012

@weierophinney weierophinney Merge branch 'hotfix/2932' into develop
Forward port #2932
ce33449

@weierophinney weierophinney added a commit to zendframework/zend-session that referenced this pull request May 15, 2015

@weierophinney weierophinney [zendframework/zendframework#2932] different save_handler in tests
- "sqlite" save handler is not installed by default; changed test to use
  "user" save handler instead
e1daa98

@weierophinney weierophinney added a commit to zendframework/zend-session that referenced this pull request May 15, 2015

@weierophinney weierophinney Merge branch 'hotfix/2932' 441f51a

@weierophinney weierophinney added a commit to zendframework/zend-session that referenced this pull request May 15, 2015

@weierophinney weierophinney Merge branch 'hotfix/2932' into develop 26ed46e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment