Skip to content

Commit 60a822a

Browse files
committed
Remember to check for is_authenticated() before accessing
get_profile() on user - Legacy-Id: 3363
1 parent 1cc908f commit 60a822a

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

ietf/liaisons/accountsREDESIGN.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,9 @@ def get_iab_executive_director():
3131

3232

3333
def get_person_for_user(user):
34+
if not user.is_authenticated():
35+
return None
36+
3437
p = user.get_profile()
3538
p.email = lambda: (p.name, p.email_address().address)
3639

@@ -83,7 +86,7 @@ def is_sdo_authorized_individual(person):
8386

8487

8588
def is_secretariat(user):
86-
return bool(Role.objects.filter(email__person=user.get_profile(), name="secr", group__acronym="secretariat"))
89+
return user.is_authenticated() and bool(Role.objects.filter(email__person=user.get_profile(), name="secr", group__acronym="secretariat"))
8790

8891

8992
def can_add_incoming_liaison(user):

0 commit comments

Comments
 (0)