fix(integrations): make email matching case insensitive (#20827)

Author: Stephen Cefali

src/sentry/api/serializers/models/release.py

@@ -78,17 +78,19 @@ def get_users_for_authors(organization_id, authors, user=None):
     # Figure out which email address matches to a user
     users_by_email = {}
     for email in user_emails:
-        if email.email not in users_by_email:
+        # force emails to lower case so we can do case insensitive matching
+        lower_email = email.email.lower()
+        if lower_email not in users_by_email:
             user = users_by_id.get(six.text_type(email.user_id), None)
             # user can be None if there's a user associated
             # with user_email in separate organization
             if user:
-                users_by_email[email.email] = user
+                users_by_email[lower_email] = user
 
     results = {}
     for author in authors:
         results[six.text_type(author.id)] = users_by_email.get(
-            author.email, {"name": author.name, "email": author.email}
+            author.email.lower(), {"name": author.name, "email": author.email}
         )
 
     return results

src/sentry/models/commitauthor.py

@@ -8,7 +8,7 @@
 
 class CommitAuthorManager(BaseManager):
     def get_or_create(self, organization_id, email, defaults, **kwargs):
-        # Force email address to lowercase because GitHub does this. Note though that this isn't technically
+        # Force email address to lowercase because many providers do this. Note though that this isn't technically
         # to spec; only the domain part of the email address is actually case-insensitive.
         # See: https://stackoverflow.com/questions/9807909/are-email-addresses-case-sensitive
         return super(CommitAuthorManager, self).get_or_create(

tests/sentry/api/serializers/test_release.py

@@ -159,7 +159,9 @@ def test_no_tag_data(self):
         assert not result["lastEvent"]
 
     def test_get_user_from_email(self):
-        user = User.objects.create(email="[email protected]")
+        user = User.objects.create(
+            email="[email protected]"
+        )  # upper case so we can test case sensitivity
         UserEmail.get_primary_email(user=user)
         project = self.create_project()
         self.create_member(user=user, organization=project.organization)