Protect against buggy participants reports from Aalto SISU API

Fixes #1180

Author: ihalaij1

course/models.py

@@ -24,6 +24,7 @@
 from django.utils.text import format_lazy
 from django.utils.translation import gettext_lazy as _
 from django_colortag.models import ColorTag
+from requests.exceptions import HTTPError
 
 from apps.models import BaseTab, BasePlugin
 from authorization.models import JWTAccessible
@@ -935,6 +936,11 @@ def enroll_from_sis(self) -> Tuple[int, int]:
         delcount = 0
         try:
             participants = sis.get_participants(self.sis_id)
+        except HTTPError as exc:
+            code = exc.response.status_code
+            # pylint: disable-next=logging-fstring-interpolation
+            logger.exception(f"{self}: Error {code} when getting participants from SIS.")
+            return -1, -1
         except Exception:
             # pylint: disable-next=logging-fstring-interpolation
             logger.exception(f"{self}: Error in getting participants from SIS.")
@@ -954,16 +960,18 @@ def enroll_from_sis(self) -> Tuple[int, int]:
                 # yet logged in to A+, then the user profile does not exist yet.
                 pass
 
-        # Remove SIS-enrolled students who are not anymore in SIS participants,
-        # for example, because they have first enrolled in SIS, but then
-        # unenrolled themselves.
-        students = self.all_students.filter(enrollment__from_sis=True)
-        to_remove = students.exclude(student_id__in=participants)
-        qs = Enrollment.objects.filter(user_profile__in=to_remove, course_instance=self)
-        qs.update(status=Enrollment.ENROLLMENT_STATUS.REMOVED)
-        for e in qs:
-            invalidate_content(Enrollment, e)
-            delcount += 1
+        # Ignore empty participants list caused by a rare SIS API gateway malfunction
+        if participants:
+            # Remove SIS-enrolled students who are not anymore in SIS participants,
+            # for example, because they have first enrolled in SIS, but then
+            # unenrolled themselves.
+            students = self.all_students.filter(enrollment__from_sis=True)
+            to_remove = students.exclude(student_id__in=participants)
+            qs = Enrollment.objects.filter(user_profile__in=to_remove, course_instance=self)
+            qs.update(status=Enrollment.ENROLLMENT_STATUS.REMOVED)
+            for e in qs:
+                invalidate_content(Enrollment, e)
+                delcount += 1
 
         # pylint: disable-next=logging-fstring-interpolation
         logger.info(f"{self}: enrolled {addcount}, removed {delcount} students based on SIS")