Skip to content

Commit 398a762

Browse files
authored
Catch errors hitting Keycloak API when attaching users to orgs (#3039)
1 parent f8fe993 commit 398a762

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

b2b/models.py

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
from django.utils.text import slugify
1111
from mitol.common.models import TimestampedModel
1212
from mitol.common.utils import now_in_utc
13+
from requests.exceptions import HTTPError
1314
from wagtail.admin.panels import FieldPanel, MultiFieldPanel
1415
from wagtail.fields import RichTextField
1516
from wagtail.models import Page
@@ -115,7 +116,7 @@ def get_learners(self):
115116
return (
116117
get_user_model()
117118
.objects.filter(
118-
b2b_contracts__organization=self,
119+
b2b_organizations=self,
119120
)
120121
.distinct()
121122
)
@@ -132,7 +133,13 @@ def attach_user(self, user):
132133

133134
from b2b.api import add_user_org_membership
134135

135-
return add_user_org_membership(self, user)
136+
try:
137+
return add_user_org_membership(self, user)
138+
except HTTPError:
139+
log.exception(
140+
"Got HTTP error attempting to attach %s to org %s, skipping", user, self
141+
)
142+
return False
136143

137144
def add_user_contracts(self, user):
138145
"""

0 commit comments

Comments
 (0)