diff --git a/edx_proctoring/__init__.py b/edx_proctoring/__init__.py index 6c0236765c0..56ecc9f16f8 100644 --- a/edx_proctoring/__init__.py +++ b/edx_proctoring/__init__.py @@ -5,6 +5,6 @@ from __future__ import absolute_import # Be sure to update the version number in edx_proctoring/package.json -__version__ = '2.3.2' +__version__ = '2.3.3' default_app_config = 'edx_proctoring.apps.EdxProctoringConfig' # pylint: disable=invalid-name diff --git a/edx_proctoring/api.py b/edx_proctoring/api.py index b0f9e05700d..f3483313049 100644 --- a/edx_proctoring/api.py +++ b/edx_proctoring/api.py @@ -690,6 +690,20 @@ def create_exam_attempt(exam_id, user_id, taking_as_proctored=False): ) log.error(log_msg) + if not external_id and exam['backend'] == 'proctortrack' and taking_as_proctored: + # temporary logging added to better understand cases where we end up + # with exam attempts with NULL external_ids - a situation, that, theoretically, + # shouldn't be occurring (EDUCATOR-4948) + log_msg = ( + u'Registered exam attempt with an external ID {external_id}' + u'in {exam_id} for user {user_id}'.format( + external_id=external_id, + exam_id=exam_id, + user_id=user_id, + ) + ) + log.error(log_msg) + attempt = ProctoredExamStudentAttempt.create_exam_attempt( exam_id, user_id, diff --git a/package.json b/package.json index b113a47e2af..e80f8d1cae1 100644 --- a/package.json +++ b/package.json @@ -2,7 +2,7 @@ "name": "@edx/edx-proctoring", "//": "Be sure to update the version number in edx_proctoring/__init__.py", "//": "Note that the version format is slightly different than that of the Python version when using prereleases.", - "version": "2.3.2", + "version": "2.3.3", "main": "edx_proctoring/static/index.js", "repository": { "type": "git",