mahara-contributors team mailing list archive

[Ghada El-Zoghbi <ghada@xxxxxxxxxxxxxxx>]

Please ignore!  The upgrade was wrong and didn't actually correctly
update the ini.php file which takes care of an empty sessionpath.

Sorry about this.


** Changed in: mahara
       Status: New => Invalid

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1396369

Title:
  Error upgrading from 1.8.6 to 1.9 with session directory is not
  writable

Status in Mahara ePortfolio:
  Invalid

Bug description:
  Mahara: 1.8.6 -> 1.9.4
  OS: Linux
  DB: Postgres
  Browser: Firefox

  An error occurs when upgrading (through cli) an existing site from
  1.8.6 to 1.9.4:

  [WAR] a5 (lib/mahara.php:158) Your session data directory, , is not writable. Please create the directory if it does not exist or give ownership of the directory to the web server user if it does.
  Call stack (most recent first):
    * ensure_sanity() at /var/www/acttqi/mahara/htdocs/init.php:97
    * require("/var/www/acttqi/mahara/htdocs/init.php") at /var/www/acttqi/mahara/htdocs/admin/cli/upgrade.php:19

  The same error also occurs when viewing in a browser.

  And, is triggerred from mahara.php::ensure_sanity().

  
  The issue is because config.php doesn't contain:

  $cfg->sessionpath = $cfg->dataroot . '/sessions';

  Once that is included in config.php, the user can proceed to upgrade
  the site.

  The release notes do not indicate that this paramater should be set
  before attempting to upgrade.

  But, in my opinion, if you're not moving your sessions outside of
  dataroot, you shouldn't need to set this parameter.

  The sanity check should really only verify the path exists if
  sessionpath is set.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1396369/+subscriptions