* @ingroup Cache
*/
class ObjectCacheSessionHandler {
- /** @var array Map of (session ID => SHA-1 of the data) */
- protected static $hashCache = array();
+
+ const TTL_REFRESH_WINDOW = 600; // refresh if expiring in 10 minutes
/**
* Install a session handler for the current web request
*/
protected static function getCache() {
global $wgSessionCacheType;
-
return ObjectCache::getInstance( $wgSessionCacheType );
}
return wfMemcKey( 'session', $id );
}
- /**
- * @param mixed $data
- * @return string
- */
- protected static function getHash( $data ) {
- return sha1( serialize( $data ) );
- }
-
/**
* Callback when opening a session.
*
*/
static function read( $id ) {
$data = self::getCache()->get( self::getKey( $id ) );
-
- self::$hashCache = array( $id => self::getHash( $data ) );
-
- return ( $data === false ) ? '' : $data;
+ if ( $data === false ) {
+ return '';
+ }
+ return $data;
}
/**
*/
static function write( $id, $data ) {
global $wgObjectCacheSessionExpiry;
-
- // Only issue a write if anything changed (PHP 5.6 already does this)
- if ( !isset( self::$hashCache[$id] )
- || self::getHash( $data ) !== self::$hashCache[$id]
- ) {
- self::getCache()->set( self::getKey( $id ), $data, $wgObjectCacheSessionExpiry );
- }
-
+ self::getCache()->set( self::getKey( $id ), $data, $wgObjectCacheSessionExpiry );
return true;
}
}
/**
- * Shutdown function. See the comment inside ObjectCacheSessionHandler::install
- * for rationale.
+ * Shutdown function.
+ * See the comment inside ObjectCacheSessionHandler::install for rationale.
*/
static function handleShutdown() {
+ global $wgObjectCacheSessionExpiry;
+
+ $now = microtime( true );
+ // Session are only written in object stores when $_SESSION changes,
+ // which also renews the TTL ($wgObjectCacheSessionExpiry). If a user
+ // is active but not causing session data changes, it may suddenly
+ // as they view a form, blocking the first submission.
+ // Make a dummy change every so often to avoid this.
+ if ( !isset( $_SESSION['wsExpiresUnix'] )
+ || ( $now + self::TTL_REFRESH_WINDOW ) > isset( $_SESSION['wsExpiresUnix'] )
+ ) {
+ $_SESSION['wsExpiresUnix'] = $now + $wgObjectCacheSessionExpiry;
+ }
+
session_write_close();
}
}