temp: add temporary exception handling in course and course run apis …

…to log the complete error (#4167) * temp: add temporary exception handling in course and course run apis to log the complete error
openedx · Oct 30, 2023 · 4e9386c · 4e9386c
1 parent bdace1f
commit 4e9386c
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/course_discovery/apps/api/v1/views/course_runs.py b/course_discovery/apps/api/v1/views/course_runs.py
@@ -288,7 +288,13 @@ def _update_course_run(self, course_run, draft, changed, serializer, request, pr
         if not draft and (course_run.status == CourseRunStatus.Unpublished or non_exempt_update):
             save_kwargs['status'] = CourseRunStatus.LegalReview
 
-        course_run = serializer.save(**save_kwargs)
+        try:
+            course_run = serializer.save(**save_kwargs)
+        except Exception:
+            log.exception(
+                f"Exception raised when attempting to save course run {course_run.key} with arguments {save_kwargs}"
+            )
+            raise  # re-raise the exception so that it is captured by writable_request_wrapper
 
         if course_run in course_run.course.active_course_runs:
             course_run.update_or_create_seats(course_run.type, prices, upgrade_deadline_override,)

diff --git a/course_discovery/apps/api/v1/views/courses.py b/course_discovery/apps/api/v1/views/courses.py
@@ -395,8 +395,14 @@ def update_course(self, data, partial=False):  # pylint: disable=too-many-statem
                     _('Course edit was unsuccessful. The course URL slug ‘[{url_slug}]’ is already in use. '
                       'Please update this field and try again.').format(url_slug=url_slug))
 
-        # Then the course itself
-        course = serializer.save()
+        try:
+            # Then the course itself
+            course = serializer.save()
+        except Exception:
+            logger.exception(
+                f"Exception raised when attempting to save course {course.key} with arguments {data}"
+            )
+            raise  # re-raise the exception so that it is captured by writable_request_wrapper
         if url_slug:
             course.set_active_url_slug(url_slug)
             if course.official_version and (not draft or self._is_course_run_reviewed(course)):