We did a lot of work on the session handling in Drupal this release with the goal of making sessions pluggable. All queries on the sessions table were consolidated into sessions.inc. Chx has already provided an alternative session handling file (for memcache) in his sandbox, and other session handling strategies are sure to appear. None of them will be very useful without this patch, however, since including ./includes/session.inc is still hardcoded into bootstrap.inc.

This one-line patch uses variable_get to get the path for the session.inc file, just like we do for cache.inc, so that an administrator can override the path in the $conf array in settings.php. This completes the effort to make Drupal session handling completely pluggable.

CommentFileSizeAuthor
bootstrap_21.patch670 bytesrobertDouglass
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

killes@www.drop.org’s picture

I am in favour of this, doesn't break anything.

chx’s picture

Status: Needs review » Reviewed & tested by the community

I obviously like it...

Steven’s picture

Status: Reviewed & tested by the community » Fixed

Given that the code is identical to cache.inc and there are no side-effects to this patch, I'm allowing it in.

Committed to HEAD :).

Anonymous’s picture

Status: Fixed » Closed (fixed)