mahara-contributors team mailing list archive

[Robert Lyon <robertl@xxxxxxxxxxxxxxx>]

There is also the problem with the headers sent error - the problem
happens after LDAP sync finishes and is happening due to this:

Once the sync_users script has run the last step is to log out current
user so they are forced to login to see any changes. Part of the logout
function is to set a cookie and this can't be done after headers already
sent.

I've added a fix to the patch handle the setcookie function to be skipped if headers already set
https://reviews.mahara.org/#/c/9137/

-- 
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/1792638

Title:
  Problems with ldap user sync database error

Status in Mahara:
  In Progress
Status in Mahara 17.04 series:
  Confirmed
Status in Mahara 17.10 series:
  Confirmed
Status in Mahara 18.04 series:
  Confirmed
Status in Mahara 18.10 series:
  In Progress

Bug description:
  When trying to use the LDAP user_sync() function there is this error

  [DBG] a0 (lib/dml.php:157) mysqli error: [1171: All parts of a PRIMARY KEY must be NOT NULL; if you need NULL in a key, use UNIQUE instead] in EXECUTE("CREATE TEMPORARY TABLE auth_ldap_extusers_temp (
  [DBG] a0 (lib/dml.php:157) extusername VARCHAR(64) DEFAULT NULL,
  [DBG] a0 (lib/dml.php:157) firstname TEXT,
  [DBG] a0 (lib/dml.php:157) lastname TEXT,
  [DBG] a0 (lib/dml.php:157) email VARCHAR(255) DEFAULT NULL,
  [DBG] a0 (lib/dml.php:157) studentid TEXT,
  [DBG] a0 (lib/dml.php:157) preferredname TEXT,
  [DBG] a0 (lib/dml.php:157) CONSTRAINT PRIMARY KEY (extusername)
  [DBG] a0 (lib/dml.php:157) )ENGINE=innodb")

  It looks like we need to make the 'extusername' column a not null
  column

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