Skip to content

Commit a4fd2d6

Browse files
committed
Don't send heavy queries anymore
1 parent d233d04 commit a4fd2d6

File tree

1 file changed

+16
-13
lines changed

1 file changed

+16
-13
lines changed

pgcommitfest/commitfest/lookups.py

Lines changed: 16 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -20,20 +20,23 @@ def userlookup(request):
2020
| Q(last_name__icontains=query),
2121
)
2222

23+
if not request.user.is_authenticated:
24+
return HttpResponseForbidden("Login required when not filtering by commitfest")
25+
_ = cf
2326
# If no commitfest filter is provided, require login
24-
if not cf:
25-
if not request.user.is_authenticated:
26-
return HttpResponseForbidden(
27-
"Login required when not filtering by commitfest"
28-
)
29-
else:
30-
# Filter users to only those who have participated in the specified commitfest
31-
# This includes authors, reviewers, and committers of patches in that commitfest
32-
users = users.filter(
33-
Q(patch_author__commitfests__id=cf)
34-
| Q(patch_reviewer__commitfests__id=cf)
35-
| Q(committer__patch__commitfests__id=cf)
36-
).distinct()
27+
# if not cf:
28+
# if not request.user.is_authenticated:
29+
# return HttpResponseForbidden(
30+
# "Login required when not filtering by commitfest"
31+
# )
32+
# else:
33+
# # Filter users to only those who have participated in the specified commitfest
34+
# # This includes authors, reviewers, and committers of patches in that commitfest
35+
# users = users.filter(
36+
# Q(patch_author__commitfests__id=cf)
37+
# | Q(patch_reviewer__commitfests__id=cf)
38+
# | Q(committer__patch__commitfests__id=cf)
39+
# ).distinct()
3740

3841
return HttpResponse(
3942
json.dumps(

0 commit comments

Comments
 (0)