allow sessions to file. fixes #2001, #2186 and refs #3454 #3522
Conversation
craigh
commented
Apr 5, 2017
•
craigh
commented
| Q | A |
|---|---|
| Bug fix? | yes |
| New feature? | no |
| BC breaks? | no |
| Deprecations? | no |
| Fixed tickets | fixes #2001, #2186 |
| Refs tickets | #3454 |
| License | MIT |
| Changelog updated | yes |
|
hmm - a problem with the CLI invalid csrf ... 🤔 😠 @matthiasnoback - do you know why symfony-console-form is using the tokens? I thought it would ignore them? see https://travis-ci.org/zikula/core/builds/219052481 |
|
well I came up with a sort of hacky fix to the csrf problem when using cli. |
src/docs/Core-2.0/Session.md
Outdated
| =================== | ||
|
|
||
| Sessions can be stored in the Database or in the Filesystem. This choice as well as a few configuration options | ||
| can be set in the ZikulaSecurityModule: |
There was a problem hiding this comment.
Typo. How about Zikula SecurityCenter module?
| $request->getSession()->getFlashBag()->add('info', $this->__f( | ||
| 'Session are currently configured to use the filesystem. It is recommended that you change this to use the database. Click %here% to configure.', | ||
| ['%here%' => '<a href="' . $router->generate('zikulasecuritycentermodule_config_config') . '">' . $this->__('Security Center') . '</a>'] | ||
| )); |
There was a problem hiding this comment.
Would it be possible to change it to the database automatically during/after the CompleteStage?
There was a problem hiding this comment.
it forces a re-login, so I didn't think that was the best method.
|
@craigh Indeed, I thought the |
|
@matthiasnoback - we are using an older version of your library with this branch - could that be the cause? Was there anything special with the form classes I was to do? |
|
@craigh Honestly, don't know. |
|
@matthiasnoback looks like I just needed to update the library. I was using an older version (v1) because this branch originally was restricted to php >= 5.4, but now it is updated to require php >=5.5 and so I can use v2 of the lib and this fixes the problem. |
