Merge pull request #174 from USEPA/fix_host_bug

dont use host header directly use get_host since referer is used.
USEPA · Sep 24, 2024 · 68f4bc0 · 68f4bc0
2 parents aacd4c5 + 7080523
commit 68f4bc0
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/accounts/admin.py b/accounts/admin.py
@@ -136,6 +136,7 @@ def has_add_permission(self, request):
     def has_delete_permission(self, request, obj=None):
         return False
 
+    # this may no longer be needed due to using autcomplete fields
     def get_search_results(self, request, queryset, search_term):
         referrer = request.META.get('HTTP_REFERER', '')
         if 'responseproject' in referrer:

diff --git a/accounts/func.py b/accounts/func.py
@@ -96,7 +96,7 @@ def enable_account(account_request, password):
 
 def get_object_id_from_request(request):
     referrer = request.META.get('HTTP_REFERER')
-    host = request.META.get('HTTP_HOST')
+    host = request.get_host()
     path = referrer.split(host)[1].split('?')[0]
     r = resolve(path)
     return r.kwargs.get('object_id', None)