Merge pull request #2039 from MIT-LCP/username-type-error

Fix server error if username is none
MIT-LCP · Jul 5, 2023 · 303edbb · 303edbb
2 parents 6f3b623 + f5dbf5a
commit 303edbb
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/physionet-django/user/backends.py b/physionet-django/user/backends.py
@@ -12,6 +12,10 @@ class DualAuthModelBackend(ModelBackend):
     This is a ModelBacked that allows authentication with either a username or an email address.
     """
     def authenticate(self, request, username=None, password=None):
+        # After implementation of OAuth2, (presumably) many crawlers started to attempt
+        # authentication with tokens, causing server errors due to the comparison below.
+        if username is None:
+            return None
         if '@' in username:
             kwargs = {'email': username.lower()}
         else: