Skip to content

Commit d30981d

Browse files
committed
Merged in [9519] from rcross@amsl.com:
Fixes issue ietf-tools#1676. Fixes ietf-tools#1676. RelatedObjectDoesNotExist when User has no Person. - Legacy-Id: 9538 Note: SVN reference [9519] has been migrated to Git commit 80d6564
2 parents f1e7a09 + 80d6564 commit d30981d

2 files changed

Lines changed: 7 additions & 2 deletions

File tree

ietf/secr/utils/decorators.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
from django.conf import settings
44
from django.contrib.auth import REDIRECT_FIELD_NAME
5+
from django.core.exceptions import ObjectDoesNotExist
56
from django.http import HttpResponseRedirect, HttpResponseForbidden
67
from django.shortcuts import render_to_response, get_object_or_404
78
from django.utils.http import urlquote
@@ -65,7 +66,11 @@ def wrapper(request, *args, **kwargs):
6566
session = slide.session_set.all()[0]
6667
group = session.group
6768

68-
login = request.user.person
69+
try:
70+
login = request.user.person
71+
except ObjectDoesNotExist:
72+
return HttpResponseForbidden("User not authorized to access group: %s" % group.acronym)
73+
6974
groups = [group]
7075
if group.parent:
7176
groups.append(group.parent)

ready-for-merge

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
# --- Add entries at the top ---
99

10-
personal/lars/6.0.2.dev0@9526
10+
personal/lars/6.0.2.dev0@9523:9526
1111
personal/willem/v5.12.4.dev0@9316
1212
branch/iola/facelift-r8876@8970 Test crawler speed-up
1313
branch/iola/facelift-r8510@8733

0 commit comments

Comments
 (0)