[SessionManagerFactory] Configuration of validators in SessionManagerFactory #4995

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants
Contributor

Blinden commented Aug 21, 2013

This is merely a cut and past from http://framework.zend.com/manual/2.2/en/modules/zend.session.manager.html to add handling of validators in the SessionManagerFactory

If this is a good addition I will write the tests

Owner

weierophinney commented Aug 21, 2013

Please add a unit test for this functionality.

Contributor

mwillbanks commented Oct 9, 2013

Failures in the build are not due to this change. Looking more into the change.

Contributor

mwillbanks commented Oct 9, 2013

We may want to create this as a separate service; such that it would be a SessionValidatorService which could take an array of options that could provide the validators and attach. That would be the only area that this might need to scale out for.

Owner

weierophinney commented Oct 22, 2013

@mwillbanks Considering this is a composed object in the SessionManager, I think having it in the same factory is fine. Merging for 2.3.0.

@ghost ghost assigned weierophinney Oct 22, 2013

weierophinney added a commit that referenced this pull request Oct 22, 2013

Merge pull request #4995 from Blinden/session-manager-factory-validators
[SessionManagerFactory] Configuration of validators in SessionManagerFactory

weierophinney added a commit that referenced this pull request Oct 22, 2013

Owner

weierophinney commented Oct 22, 2013

Merged to develop for release with 2.3.0.

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

Merge pull request zendframework/zendframework#4995 from Blinden/sess…
…ion-manager-factory-validators

[SessionManagerFactory] Configuration of validators in SessionManagerFactory

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

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