Merge pull request #239 from edx/cdodge/be-sure-to-store-video-url-as-https

change over any http: links over to https:

edx_proctoring/backends/software_secure.py

@@ -191,6 +191,9 @@ class SoftwareSecureBackendProvider(ProctoringBackendProvider):
         review_status = payload['reviewStatus']
         video_review_link = payload['videoReviewLink']
 
+        # be sure to change over any http: to https: on the video_review_link
+        video_review_link = video_review_link.replace('http:', 'https:')
+
         # do we already have a review for this attempt?!? We may not allow updates
         review = ProctoredExamSoftwareSecureReview.get_review_by_attempt_code(attempt_code)
 

edx_proctoring/backends/tests/test_software_secure.py

@@ -488,7 +488,7 @@ class SoftwareSecureTests(TestCase):
         self.assertEqual(review.review_status, review_status)
         self.assertEqual(
             review.video_url,
-            'http://www.remoteproctor.com/AdminSite/Account/Reviewer/DirectLink-Generic.aspx?ID=foo'
+            'https://www.remoteproctor.com/AdminSite/Account/Reviewer/DirectLink-Generic.aspx?ID=foo'
         )
         self.assertIsNotNone(review.raw_data)
         self.assertIsNone(review.reviewed_by)
@@ -662,7 +662,7 @@ class SoftwareSecureTests(TestCase):
         self.assertEqual(review.review_status, 'Clean')
         self.assertEqual(
             review.video_url,
-            'http://www.remoteproctor.com/AdminSite/Account/Reviewer/DirectLink-Generic.aspx?ID=foo'
+            'https://www.remoteproctor.com/AdminSite/Account/Reviewer/DirectLink-Generic.aspx?ID=foo'
         )
         self.assertIsNotNone(review.raw_data)
 
@@ -760,7 +760,7 @@ class SoftwareSecureTests(TestCase):
         self.assertEqual(review.review_status, 'Suspicious')
         self.assertEqual(
             review.video_url,
-            'http://www.remoteproctor.com/AdminSite/Account/Reviewer/DirectLink-Generic.aspx?ID=foo'
+            'https://www.remoteproctor.com/AdminSite/Account/Reviewer/DirectLink-Generic.aspx?ID=foo'
         )
         self.assertIsNotNone(review.raw_data)