fix: Suggested changes

Author: KShivendu

examples/with-fastapi/with-thirdpartyemailpassword/main.py

@@ -19,6 +19,7 @@
     emailverification,
     session,
     thirdpartyemailpassword,
+    usermetadata,
 )
 from supertokens_python.recipe.session import SessionContainer
 from supertokens_python.recipe.session.framework.fastapi import verify_session
@@ -54,6 +55,7 @@ def get_website_domain():
         session.init(),
         dashboard.init(),
         emailverification.init("REQUIRED"),
+        usermetadata.init(),
         thirdpartyemailpassword.init(
             providers=[
                 thirdpartyemailpassword.ProviderInput(

supertokens_python/recipe/dashboard/utils.py

@@ -88,6 +88,10 @@ def from_user(
         self.last_name = last_name
 
         self.user_id = user.user_id
+        # from_user() is called in /api/users (note extra s)
+        # here we DashboardUsersGetResponse() doesn't maintain
+        # recipe id for each user on its own. That's why we need
+        # to set self.recipe_id here.
         self.recipe_id = user.recipe_id
         self.time_joined = user.time_joined
         self.email = user.email
@@ -109,7 +113,11 @@ def from_dict(
         self.last_name = last_name
 
         self.user_id = user_obj_dict["user_id"]
-        # self.recipe_id = user_obj_dict.get("recipe_id")
+        # from_dict() is used in `/api/user` where
+        # recipe_id is already passed seperately to
+        # GetUserForRecipeIdResult object
+        # So we set recipe_id to None here
+        self.recipe_id = None
         self.time_joined = user_obj_dict["time_joined"]
         self.tenant_ids = user_obj_dict.get("tenant_ids", [])
 

supertokens_python/recipe/session/framework/fastapi/__init__.py

@@ -43,8 +43,6 @@ def verify_session(
 ) -> Callable[..., Coroutine[Any, Any, Union[SessionContainer, None]]]:
     _ = user_context
 
-    from fastapi import Request  # pylint: disable=reimported
-
     async def func(request: Request) -> Union[SessionContainer, None]:
         nonlocal user_context
         base_req = FastApiRequest(request)

supertokens_python/recipe/thirdparty/providers/boxy_saml.py

@@ -56,8 +56,6 @@ def BoxySAML(input: ProviderInput) -> Provider:  # pylint: disable=redefined-bui
     if input.config.user_info_map is None:
         input.config.user_info_map = UserInfoMap(UserFields(), UserFields())
 
-    if input.config.user_info_map.from_user_info_api is None:
-        input.config.user_info_map.from_user_info_api = UserFields()
     if input.config.user_info_map.from_id_token_payload is None:
         input.config.user_info_map.from_id_token_payload = UserFields()
 

supertokens_python/recipe/thirdparty/providers/discord.py

@@ -58,8 +58,6 @@ def Discord(input: ProviderInput) -> Provider:  # pylint: disable=redefined-buil
 
     if input.config.user_info_map.from_user_info_api is None:
         input.config.user_info_map.from_user_info_api = UserFields()
-    if input.config.user_info_map.from_id_token_payload is None:
-        input.config.user_info_map.from_id_token_payload = UserFields()
 
     if input.config.user_info_map.from_user_info_api.user_id is None:
         input.config.user_info_map.from_user_info_api.user_id = "id"

supertokens_python/recipe/thirdparty/providers/facebook.py

@@ -75,8 +75,6 @@ def Facebook(input: ProviderInput) -> Provider:  # pylint: disable=redefined-bui
 
     if input.config.user_info_map.from_user_info_api is None:
         input.config.user_info_map.from_user_info_api = UserFields()
-    if input.config.user_info_map.from_id_token_payload is None:
-        input.config.user_info_map.from_id_token_payload = UserFields()
 
     if input.config.user_info_map.from_user_info_api.user_id is None:
         input.config.user_info_map.from_user_info_api.user_id = "id"

supertokens_python/recipe/thirdparty/providers/linkedin.py

@@ -105,8 +105,6 @@ def Linkedin(input: ProviderInput) -> Provider:  # pylint: disable=redefined-bui
 
     if input.config.user_info_map.from_user_info_api is None:
         input.config.user_info_map.from_user_info_api = UserFields()
-    if input.config.user_info_map.from_id_token_payload is None:
-        input.config.user_info_map.from_id_token_payload = UserFields()
 
     if input.config.user_info_map.from_user_info_api.user_id is None:
         input.config.user_info_map.from_user_info_api.user_id = "id"