Merge pull request #402 from supertokens/fix/tp-rework-user-password-put

fix: Fix tenant id being passed in the wrong order for password update functions

Author: rishabhpoddar

supertokens_python/recipe/dashboard/api/userdetails/user_password_put.py

@@ -106,7 +106,7 @@ async def reset_password(
             )
 
         password_reset_token = await create_reset_password_token(
-            user_id, tenant_id, user_context
+            tenant_id, user_id, user_context
         )
 
         if isinstance(password_reset_token, CreateResetPasswordWrongUserIdError):
@@ -115,7 +115,7 @@ async def reset_password(
             raise Exception("Should never come here")
 
         password_reset_response = await reset_password_using_token(
-            password_reset_token.token, new_password, tenant_id, user_context
+            tenant_id, password_reset_token.token, new_password, user_context
         )
 
         if isinstance(

supertokens_python/recipe/multitenancy/api/implementation.py

@@ -40,12 +40,16 @@ async def login_methods_get(
         api_options: APIOptions,
         user_context: Dict[str, Any],
     ) -> Union[LoginMethodsGetOkResult, GeneralErrorResponse]:
-        tenant_config_res = await api_options.recipe_implementation.get_tenant(
+
+        tenant_config = await api_options.recipe_implementation.get_tenant(
             tenant_id, user_context
         )
 
+        if tenant_config is None:
+            raise Exception("Tenant not found")
+
         provider_inputs_from_static = api_options.static_third_party_providers
-        provider_configs_from_core = tenant_config_res.third_party.providers
+        provider_configs_from_core = tenant_config.third_party.providers
 
         merged_providers = merge_providers_from_core_and_static(
             provider_configs_from_core, provider_inputs_from_static
@@ -61,24 +65,22 @@ async def login_methods_get(
                     client_type,
                     user_context,
                 )
-                final_provider_list.append(
-                    ThirdPartyProvider(
-                        provider_instance.id, provider_instance.config.name
-                    )
-                )
-            except Exception as e:
-                if isinstance(e, ClientTypeNotFoundError):
-                    continue
-                raise e
+
+                if provider_instance is None:
+                    raise Exception("Should never come here")
+
+            except ClientTypeNotFoundError:
+                continue
+            final_provider_list.append(
+                ThirdPartyProvider(provider_instance.id, provider_instance.config.name)
+            )
 
         return LoginMethodsGetOkResult(
             email_password=LoginMethodEmailPassword(
-                tenant_config_res.emailpassword.enabled
-            ),
-            passwordless=LoginMethodPasswordless(
-                tenant_config_res.passwordless.enabled
+                tenant_config.emailpassword.enabled
             ),
+            passwordless=LoginMethodPasswordless(tenant_config.passwordless.enabled),
             third_party=LoginMethodThirdParty(
-                tenant_config_res.third_party.enabled, final_provider_list
+                tenant_config.third_party.enabled, final_provider_list
             ),
         )

supertokens_python/recipe/multitenancy/asyncio/__init__.py

@@ -61,7 +61,7 @@ async def delete_tenant(
 
 async def get_tenant(
     tenant_id: str, user_context: Optional[Dict[str, Any]] = None
-) -> GetTenantOkResult:
+) -> Optional[GetTenantOkResult]:
     if user_context is None:
         user_context = {}
     recipe = MultitenancyRecipe.get_instance()

supertokens_python/recipe/multitenancy/interfaces.py

@@ -216,7 +216,7 @@ async def delete_tenant(
     @abstractmethod
     async def get_tenant(
         self, tenant_id: str, user_context: Dict[str, Any]
-    ) -> GetTenantOkResult:
+    ) -> Optional[GetTenantOkResult]:
         pass
 
     @abstractmethod

supertokens_python/recipe/multitenancy/recipe.py

@@ -27,7 +27,6 @@
 from ...post_init_callbacks import PostSTInitCallbacks
 
 from .interfaces import (
-    APIInterface,
     APIOptions,
     TypeGetAllowedDomainsForTenantId,
 )
@@ -42,19 +41,12 @@
 
 from supertokens_python.normalised_url_path import NormalisedURLPath
 from supertokens_python.querier import Querier
-from supertokens_python.types import GeneralErrorResponse
+from supertokens_python.recipe.multitenancy.api.implementation import APIImplementation
 
 
 from .api import handle_login_methods_api
 from .constants import LOGIN_METHODS
 from .exceptions import MultitenancyError
-from .interfaces import (
-    LoginMethodsGetOkResult,
-    ThirdPartyProvider,
-    LoginMethodEmailPassword,
-    LoginMethodPasswordless,
-    LoginMethodThirdParty,
-)
 from .utils import (
     InputOverrideConfig,
     validate_and_normalise_user_input,
@@ -196,58 +188,6 @@ def reset():
         MultitenancyRecipe.__instance = None
 
 
-class APIImplementation(APIInterface):
-    async def login_methods_get(
-        self,
-        tenant_id: str,
-        client_type: Optional[str],
-        api_options: APIOptions,
-        user_context: Dict[str, Any],
-    ) -> Union[LoginMethodsGetOkResult, GeneralErrorResponse]:
-        from supertokens_python.recipe.thirdparty.providers.config_utils import (
-            find_and_create_provider_instance,
-            merge_providers_from_core_and_static,
-        )
-        from supertokens_python.recipe.thirdparty.exceptions import (
-            ClientTypeNotFoundError,
-        )
-
-        tenant_config = await api_options.recipe_implementation.get_tenant(
-            tenant_id, user_context
-        )
-
-        provider_inputs_from_static = api_options.static_third_party_providers
-        provider_configs_from_core = tenant_config.third_party.providers
-
-        merged_providers = merge_providers_from_core_and_static(
-            provider_configs_from_core, provider_inputs_from_static
-        )
-
-        final_provider_list: List[ThirdPartyProvider] = []
-
-        for provider_input in merged_providers:
-            try:
-                provider_instance = await find_and_create_provider_instance(
-                    merged_providers,
-                    provider_input.config.third_party_id,
-                    client_type,
-                    user_context,
-                )
-            except ClientTypeNotFoundError:
-                continue
-            final_provider_list.append(
-                ThirdPartyProvider(provider_instance.id, provider_instance.config.name)
-            )
-
-        return LoginMethodsGetOkResult(
-            LoginMethodEmailPassword(tenant_config.emailpassword.enabled),
-            LoginMethodPasswordless(tenant_config.passwordless.enabled),
-            LoginMethodThirdParty(
-                tenant_config.third_party.enabled, final_provider_list
-            ),
-        )
-
-
 class AllowedDomainsClaimClass(PrimitiveArrayClaim[List[str]]):
     def __init__(self):
         default_max_age_in_sec = 60 * 60

supertokens_python/recipe/multitenancy/recipe_implementation.py

@@ -159,11 +159,14 @@ async def delete_tenant(
 
     async def get_tenant(
         self, tenant_id: Optional[str], user_context: Dict[str, Any]
-    ) -> GetTenantOkResult:
+    ) -> Optional[GetTenantOkResult]:
         res = await self.querier.send_get_request(
             NormalisedURLPath(f"{tenant_id}/recipe/multitenancy/tenant"),
         )
 
+        if res["status"] == "TENANT_NOT_FOUND_ERROR":
+            return None
+
         tenant_config = parse_tenant_config(res)
 
         return GetTenantOkResult(

supertokens_python/recipe/thirdparty/api/authorisation_url.py

@@ -19,7 +19,7 @@
 if TYPE_CHECKING:
     from supertokens_python.recipe.thirdparty.interfaces import APIOptions, APIInterface
 
-from supertokens_python.exceptions import raise_bad_input_exception
+from supertokens_python.exceptions import raise_bad_input_exception, BadInputError
 from supertokens_python.utils import send_200_response
 
 
@@ -53,7 +53,12 @@ async def handle_authorisation_url_api(
         user_context=user_context,
     )
 
-    provider = provider_response.provider
+    if provider_response is None:
+        raise BadInputError(
+            f"the provider {third_party_id} could not be found in the configuration"
+        )
+
+    provider = provider_response
     result = await api_implementation.authorisation_url_get(
         provider=provider,
         redirect_uri_on_provider_dashboard=redirect_uri_on_provider_dashboard,

supertokens_python/recipe/thirdparty/api/signinup.py

@@ -19,7 +19,7 @@
 if TYPE_CHECKING:
     from supertokens_python.recipe.thirdparty.interfaces import APIOptions, APIInterface
 
-from supertokens_python.exceptions import raise_bad_input_exception
+from supertokens_python.exceptions import raise_bad_input_exception, BadInputError
 from supertokens_python.utils import send_200_response
 
 
@@ -64,7 +64,12 @@ async def handle_sign_in_up_api(
         user_context=user_context,
     )
 
-    provider = provider_response.provider
+    if provider_response is None:
+        raise BadInputError(
+            f"the provider {third_party_id} could not be found in the configuration"
+        )
+
+    provider = provider_response
 
     result = await api_implementation.sign_in_up_post(
         provider=provider,

supertokens_python/recipe/thirdparty/interfaces.py

@@ -114,7 +114,7 @@ async def get_provider(
         client_type: Optional[str],
         tenant_id: str,
         user_context: Dict[str, Any],
-    ) -> GetProviderOkResult:
+    ) -> Optional[Provider]:
         pass
 
 

supertokens_python/recipe/thirdparty/providers/config_utils.py

@@ -262,13 +262,11 @@ async def find_and_create_provider_instance(
     third_party_id: str,
     client_type: Optional[str],
     user_context: Dict[str, Any],
-) -> Provider:
+) -> Optional[Provider]:
     for provider_input in providers:
         if provider_input.config.third_party_id == third_party_id:
             provider_instance = create_provider(provider_input)
             await fetch_and_set_config(provider_instance, client_type, user_context)
             return provider_instance
 
-    raise Exception(
-        f"the provider {third_party_id} could not be found in the configuration"
-    )
+    return None