Comment 2 for bug 1792638

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/