Eventlog update in upgrade taking hours to process

Bug #1694342 reported by Ghada El-Zoghbi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Medium
Cecilia Vela Gurovic
16.10
Fix Released
Medium
Unassigned
17.04
Fix Released
Medium
Unassigned
17.10
Fix Released
Medium
Cecilia Vela Gurovic

Bug Description

OS: Linux
DB: Postgres
Mahara: 16.10.4
Browser: N/A

With the latest security release for 16.10.4, the upgrade to clean the eventlog of sensitive data can take hours to process.

We have over 56K records for the 'createuser' event. Most of them are LDAP accounts so the data looks like this:

"{"username":"abc","authinstance":"3","firstname":"first","lastname":"last","password":"","email":"<email address hidden>","studentid":"1234","ctime":"2016-07-05 11:29:21","urlid":"1234","quota":"524288000","id":85}"

The upgrade updates these records to remove the '"password":""' data.

I'll try to improve on this.

Changed in mahara:
assignee: nobody → Ghada El-Zoghbi (ghada-z)
Revision history for this message
Ghada El-Zoghbi (ghada-z) wrote :

Note: it is related to this bug: https://bugs.launchpad.net/mahara/+bug/1692749

Changed in mahara:
assignee: Ghada El-Zoghbi (ghada-z) → Ilya Tregubov (ilya-u)
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "16.10_STABLE" branch: https://reviews.mahara.org/7799

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "master" branch: https://reviews.mahara.org/7801

Changed in mahara:
status: New → In Progress
importance: Undecided → Medium
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/7801
Committed: https://git.mahara.org/mahara/mahara/commit/55d084a2b83bce06610b0774a80990aee0f2e159
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 55d084a2b83bce06610b0774a80990aee0f2e159
Author: Ilya Tregubov <email address hidden>
Date: Thu Jun 1 15:34:10 2017 +1000

Bug #1694342: clean password from event_log takes long time

behatnotneeded

Change-Id: I26a13d2ef060078abd14bb21a61db147112a4100

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "17.04_STABLE" branch: https://reviews.mahara.org/7862

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/7862
Committed: https://git.mahara.org/mahara/mahara/commit/6e8fbeea486d7c10524dfc1c273b3e531c0e5f4d
Submitter: Robert Lyon (<email address hidden>)
Branch: 17.04_STABLE

commit 6e8fbeea486d7c10524dfc1c273b3e531c0e5f4d
Author: Ilya Tregubov <email address hidden>
Date: Thu Jun 1 15:34:10 2017 +1000

Bug #1694342: clean password from event_log takes long time

behatnotneeded

Change-Id: I26a13d2ef060078abd14bb21a61db147112a4100

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/7799
Committed: https://git.mahara.org/mahara/mahara/commit/b13178e7c737c7b10eb68570fd1adf533dfcb065
Submitter: Robert Lyon (<email address hidden>)
Branch: 16.10_STABLE

commit b13178e7c737c7b10eb68570fd1adf533dfcb065
Author: Ilya Tregubov <email address hidden>
Date: Thu Jun 1 13:37:49 2017 +1000

Bug #1694342: clean password from event_log takes long time

behatnotneeded

Change-Id: Ic511e25066ebd84b80118e71f70c6f867a3564b4

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.