fix(socialaccount): provider.name must be a class level property

pennersr · Aug 19, 2023 · 6e0725b · 6e0725b
1 parent 3d685a2
commit 6e0725b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/allauth/socialaccount/providers/openid_connect/provider.py b/allauth/socialaccount/providers/openid_connect/provider.py
@@ -17,9 +17,9 @@ class OpenIDConnectProvider(OAuth2Provider):
     name = "OpenID Connect"
     account_class = OpenIDConnectProviderAccount
 
-    @property
-    def name(self):
-        return self.app.name
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        self.name = self.app.name
 
     @property
     def server_url(self):

diff --git a/allauth/socialaccount/providers/saml/provider.py b/allauth/socialaccount/providers/saml/provider.py
@@ -11,6 +11,7 @@ def to_str(self):
 
 class SAMLProvider(Provider):
     id = "saml"
+    name = "SAML"
     account_class = SAMLAccount
     default_attribute_mapping = {
         "uid": [
@@ -36,9 +37,9 @@ class SAMLProvider(Provider):
         ],
     }
 
-    @property
-    def name(self):
-        return self.app.name or self.app.client_id or self.id
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        self.name = self.app.name or self.app.client_id or self.name
 
     def get_login_url(self, request, **kwargs):
         url = reverse("saml_login", kwargs={"organization_slug": self.app.client_id})