fix: drop column ProctoredExamSoftwareSecureReview.video_url (#990)

Also introduce the encrypted_video_url column on the ProctoredExamSoftwareSecureReviewHistory table so we can have proper history records

Co-authored-by: Simon Chen <[email protected]>

CHANGELOG.rst

@@ -13,6 +13,10 @@ Change Log
 
 Unreleased
 ~~~~~~~~~~
+[4.3.3] - 2021-10-29
+~~~~~~~~~~~~~~~~~~~~
+* Remove ProctoredExamSoftwareSecureReview.video_url column from database.
+
 [4.3.2] - 2021-10-28
 ~~~~~~~~~~~~~~~~~~~~
 * Remove video_url reference from ProctoredExamSoftwareSecureReview.

edx_proctoring/__init__.py

@@ -3,6 +3,6 @@
 """
 
 # Be sure to update the version number in edx_proctoring/package.json
-__version__ = '4.3.2'
+__version__ = '4.3.3'
 
 default_app_config = 'edx_proctoring.apps.EdxProctoringConfig'  # pylint: disable=invalid-name

edx_proctoring/migrations/0021_auto_20211029_1353.py

@@ -0,0 +1,26 @@
+# Generated by Django 2.2.24 on 2021-10-29 13:53
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('edx_proctoring', '0020_auto_20211028_1915'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='proctoredexamsoftwaresecurereview',
+            name='video_url',
+        ),
+        migrations.RemoveField(
+            model_name='proctoredexamsoftwaresecurereviewhistory',
+            name='video_url',
+        ),
+        migrations.AddField(
+            model_name='proctoredexamsoftwaresecurereviewhistory',
+            name='encrypted_video_url',
+            field=models.BinaryField(null=True),
+        ),
+    ]

edx_proctoring/models.py

@@ -844,6 +844,9 @@ class ProctoredExamSoftwareSecureReviewHistory(TimeStampedModel):
     # reviewing service
     raw_data = models.TextField()
 
+    # Encrypted URL for the exam video that had been reviewed
+    encrypted_video_url = models.BinaryField(null=True)
+
     # user_id of person who did the review (can be None if submitted via server-to-server API)
     reviewed_by = models.ForeignKey(USER_MODEL, null=True, related_name='+', on_delete=models.CASCADE)
 

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": "4.3.2",
+  "version": "4.3.3",
   "main": "edx_proctoring/static/index.js",
   "scripts": {
     "test": "gulp test"