fix: Make name/email lookups case-insensitive (#5972) (#6007)

* fix: Make name/email lookups case-insensitive (#5972)

Use icontains so that looking up name or email is case insensitive
Added a test

Fixes: 5972

* fix: Use __iexact not __icontains

ietf/person/tests.py

@@ -112,6 +112,14 @@ class PersonTests(TestCase):
         r = self.client.get(url)
         self.assertContains(r, person.name, status_code=200)
 
+    def test_case_insensitive(self):
+        # Case insensitive seach
+        person = PersonFactory(name="Test Person")
+        url = urlreverse("ietf.person.views.profile", kwargs={ "email_or_name": "test person"})
+        r = self.client.get(url)
+        self.assertContains(r, person.name, status_code=200)
+        self.assertNotIn('More than one person', r.content.decode())
+
     def test_person_profile_duplicates(self):
         # same Person name and email - should not show on the profile as multiple Person records
         person = PersonFactory(name="bazquux@example.com", user__email="bazquux@example.com")

ietf/person/views.py

@@ -69,11 +69,11 @@ def ajax_select2_search(request, model_name):
 
 
 def profile(request, email_or_name):
-    aliases = Alias.objects.filter(name=email_or_name)
+    aliases = Alias.objects.filter(name__iexact=email_or_name)
     persons = set(a.person for a in aliases)
 
     if '@' in email_or_name:
-        emails = Email.objects.filter(address=email_or_name)
+        emails = Email.objects.filter(address__iexact=email_or_name)
         persons.update(e.person for e in emails)
 
     persons = [p for p in persons if p and p.id]