Ticket #621 (closed Bug: wontfix)

Opened 6 months ago

Last modified 6 months ago

Session Database Driver pulls from wrong table

Reported by: Nodren Owned by: - No owner -
Priority: blocker Milestone: 2.2
Component: Libraries:Session Version: SVN HEAD
Keywords: session, database Cc:

Description

The Session Database driver was updated in r2692 to allow for a group functionality, a new paramter $this->db_table was added, but it pulls from the wrong config value.

The Config file states that $configstorage? is the storage parameter used by drivers, which in the case of the database driver should be the name of the database table, with that revision, $this->db_table uses $configname? which per the settings is meant to be the name of the session itself.

Recommended fix: in the Session_Database_Driver construct() function change:

$this->db_table = Config::item('session.name');

to this:

$this->db_table = Config::item('session.storage');

Change History

Changed 6 months ago by armen

  • status changed from new to closed
  • resolution set to wontfix

Wontfix:

As Oscar mentioned in log message of r2692 session.storage used to set non default database group. You have to use session.name for table name now.

the only problem is that kohana does not allowed underscore in session.name so I filed a ticket(#622) to fix this issue.

Note: See TracTickets for help on using tickets.