Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Patch for CakeSession php default #312

Merged
merged 1 commit into from

2 participants

@teddyzeenny

When Session Config is set to 'defaults'=>'php', the session.save_handler is always assumed 'files' , and it will overwrite the actual php.ini configuration.
This creates problems when php session is configured to use another save_handler like memcached instead of "files".

@lorenzo
Owner

Thanks! nice catch

@lorenzo lorenzo merged commit 5d8aeac into cakephp:2.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Nov 9, 2011
  1. @teddyzeenny
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 2 deletions.
  1. +1 −2  lib/Cake/Model/Datasource/CakeSession.php
View
3  lib/Cake/Model/Datasource/CakeSession.php
@@ -539,8 +539,7 @@ protected static function _defaultConfig($name) {
'cookieTimeout' => 240,
'ini' => array(
'session.use_trans_sid' => 0,
- 'session.cookie_path' => self::$path,
- 'session.save_handler' => 'files'
+ 'session.cookie_path' => self::$path
)
),
'cake' => array(
Something went wrong with that request. Please try again.