Merge pull request #1266 from openedx/michaelroytman/COSMO-595-update-retirement-coverage-fix

Clear all ProctoredExamSoftwareSecureReviewHistory instances of PII

Author: michaelroytman

CHANGELOG.rst

@@ -14,6 +14,10 @@ Change Log
 Unreleased
 ~~~~~~~~~~
 
+[5.1.1] - 2025-02-07
+~~~~~~~~~~~~~~~~~~~~
+* fixes bug in ordering of query to ensure all Software Secure history is cleared of PII via the retirement endpoint
+
 [5.1.0] - 2025-02-03
 ~~~~~~~~~~~~~~~~~~~~
 * add Software Secure review, history, and comment models to retirement endpoint

edx_proctoring/__init__.py

@@ -3,4 +3,4 @@
 """
 
 # Be sure to update the version number in edx_proctoring/package.json
-__version__ = '5.1.0'
+__version__ = '5.1.1'

edx_proctoring/views.py

@@ -2275,8 +2275,6 @@ def _retire_proctored_exam_software_secure_review_and_history_and_comment(self,
 
             # Retire reviews and review history.
             reviews = ProctoredExamSoftwareSecureReview.objects.filter(attempt_code=attempt_code)
-            review_history = ProctoredExamSoftwareSecureReviewHistory.objects.filter(attempt_code=attempt_code)
-
             for review in reviews:
                 review.encrypted_video_url = b''
                 review.raw_data = ''
@@ -2285,6 +2283,7 @@ def _retire_proctored_exam_software_secure_review_and_history_and_comment(self,
                 # and must save each review one-by-one.
                 review.save()
 
+            review_history = ProctoredExamSoftwareSecureReviewHistory.objects.filter(attempt_code=attempt_code)
             for review in review_history:
                 review.encrypted_video_url = b''
                 review.raw_data = ''

package.json

@@ -1,7 +1,7 @@
 {
   "name": "@edx/edx-proctoring",
   "//": "Note that the version format is slightly different than that of the Python version when using prereleases.",
-  "version": "5.1.0",
+  "version": "5.1.1",
   "main": "edx_proctoring/static/index.js",
   "scripts": {
     "test": "gulp test"