Merge branch 'feat/tp-rework' into fix/mt-dashboard

Author: KShivendu

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

@@ -26,8 +26,10 @@ async def handle_sessions_get(
     if user_id is None:
         raise_bad_input_exception("Missing required parameter 'userId'")
 
+    # Passing tenant id as None sets fetch_across_all_tenants to True
+    # which is what we want here.
     session_handles = await get_all_session_handles_for_user(
-        user_id, tenant_id, user_context
+        user_id, None, user_context
     )
     sessions: List[Optional[SessionInfo]] = [None for _ in session_handles]
 

supertokens_python/recipe/emailpassword/asyncio/__init__.py

@@ -22,8 +22,8 @@
     CreateResetPasswordWrongUserIdError,
     CreateResetPasswordLinkUknownUserIdError,
     CreateResetPasswordLinkOkResult,
-    CreateResetPasswordEmailOkResult,
-    CreateResetPasswordEmailUnknownUserIdError,
+    SendResetPasswordEmailOkResult,
+    SendResetPasswordEmailUnknownUserIdError,
 )
 from supertokens_python.recipe.emailpassword.utils import get_password_reset_link
 from supertokens_python.recipe.emailpassword.types import (
@@ -165,7 +165,7 @@ async def send_reset_password_email(
 ):
     link = await create_reset_password_link(user_id, tenant_id, user_context)
     if isinstance(link, CreateResetPasswordLinkUknownUserIdError):
-        return CreateResetPasswordEmailUnknownUserIdError()
+        return SendResetPasswordEmailUnknownUserIdError()
 
     user = await get_user_by_id(user_id, user_context)
     assert user is not None
@@ -179,4 +179,4 @@ async def send_reset_password_email(
         user_context,
     )
 
-    return CreateResetPasswordEmailOkResult()
+    return SendResetPasswordEmailOkResult()

supertokens_python/recipe/emailpassword/interfaces.py

@@ -66,11 +66,11 @@ class CreateResetPasswordLinkUknownUserIdError:
     pass
 
 
-class CreateResetPasswordEmailOkResult:
+class SendResetPasswordEmailOkResult:
     pass
 
 
-class CreateResetPasswordEmailUnknownUserIdError:
+class SendResetPasswordEmailUnknownUserIdError:
     pass
 
 

supertokens_python/recipe/emailverification/recipe.py

@@ -230,7 +230,7 @@ def callback():
                 PostSTInitCallbacks.add_post_init_callback(callback)
 
                 return EmailVerificationRecipe.__instance
-            return raise_general_exception(
+            raise_general_exception(
                 "Emailverification recipe has already been initialised. Please check your code for bugs."
             )
 

supertokens_python/recipe/session/asyncio/__init__.py

@@ -441,7 +441,7 @@ async def revoke_all_sessions_for_user(
 
 async def get_all_session_handles_for_user(
     user_id: str,
-    tenant_id: Optional[str],
+    tenant_id: Optional[str] = None,
     user_context: Union[None, Dict[str, Any]] = None,
 ) -> List[str]:
     if user_context is None:

supertokens_python/recipe/session/interfaces.py

@@ -219,7 +219,7 @@ async def revoke_all_sessions_for_user(
         self,
         user_id: str,
         tenant_id: str,
-        revoke_across_all_tenants: Optional[bool],
+        revoke_across_all_tenants: bool,
         user_context: Dict[str, Any],
     ) -> List[str]:
         pass
@@ -229,7 +229,7 @@ async def get_all_session_handles_for_user(
         self,
         user_id: str,
         tenant_id: str,
-        fetch_across_all_tenants: Optional[bool],
+        fetch_across_all_tenants: bool,
         user_context: Dict[str, Any],
     ) -> List[str]:
         pass

supertokens_python/recipe/session/recipe_implementation.py

@@ -330,7 +330,7 @@ async def revoke_all_sessions_for_user(
         self,
         user_id: str,
         tenant_id: Optional[str],
-        revoke_across_all_tenants: Optional[bool],
+        revoke_across_all_tenants: bool,
         user_context: Dict[str, Any],
     ) -> List[str]:
         return await session_functions.revoke_all_sessions_for_user(
@@ -341,7 +341,7 @@ async def get_all_session_handles_for_user(
         self,
         user_id: str,
         tenant_id: Optional[str],
-        fetch_across_all_tenants: Optional[bool],
+        fetch_across_all_tenants: bool,
         user_context: Dict[str, Any],
     ) -> List[str]:
         return await session_functions.get_all_session_handles_for_user(

supertokens_python/recipe/session/session_functions.py

@@ -392,7 +392,7 @@ async def revoke_all_sessions_for_user(
     recipe_implementation: RecipeImplementation,
     user_id: str,
     tenant_id: Optional[str],
-    revoke_across_all_tenants: Optional[bool],
+    revoke_across_all_tenants: bool,
 ) -> List[str]:
     if tenant_id is None:
         tenant_id = DEFAULT_TENANT_ID
@@ -408,7 +408,7 @@ async def get_all_session_handles_for_user(
     recipe_implementation: RecipeImplementation,
     user_id: str,
     tenant_id: Optional[str],
-    fetch_across_all_tenants: Optional[bool],
+    fetch_across_all_tenants: bool,
 ) -> List[str]:
     if tenant_id is None:
         tenant_id = DEFAULT_TENANT_ID

supertokens_python/recipe/thirdpartyemailpassword/asyncio/__init__.py

@@ -25,9 +25,8 @@
     CreateResetPasswordWrongUserIdError,
     CreateResetPasswordLinkUknownUserIdError,
     CreateResetPasswordLinkOkResult,
-    CreateResetPasswordEmailUnknownUserIdError,
-    CreateResetPasswordEmailOkResult,
-    RawUserInfoFromProvider,
+    SendResetPasswordEmailUnknownUserIdError,
+    SendResetPasswordEmailEmailOkResult
 )
 from supertokens_python.recipe.emailpassword.utils import get_password_reset_link
 
@@ -63,29 +62,6 @@ async def get_user_by_third_party_info(
     )
 
 
-async def thirdparty_sign_in_up(
-    third_party_id: str,
-    third_party_user_id: str,
-    email: str,
-    oauth_tokens: Dict[str, Any],
-    raw_user_info_from_provider: RawUserInfoFromProvider,
-    tenant_id: Optional[str] = None,
-    user_context: Optional[Dict[str, Any]] = None,
-):
-    if user_context is None:
-        user_context = {}
-
-    return await ThirdPartyEmailPasswordRecipe.get_instance().recipe_implementation.thirdparty_sign_in_up(
-        third_party_id,
-        third_party_user_id,
-        email,
-        oauth_tokens,
-        raw_user_info_from_provider,
-        tenant_id or DEFAULT_TENANT_ID,
-        user_context,
-    )
-
-
 async def thirdparty_manually_create_or_update_user(
     third_party_id: str,
     third_party_user_id: str,
@@ -245,7 +221,7 @@ async def send_reset_password_email(
 ):
     link = await create_reset_password_link(user_id, tenant_id, user_context)
     if isinstance(link, CreateResetPasswordLinkUknownUserIdError):
-        return CreateResetPasswordEmailUnknownUserIdError()
+        return SendResetPasswordEmailUnknownUserIdError()
 
     user = await get_user_by_id(user_id, user_context)
     assert user is not None
@@ -259,4 +235,4 @@ async def send_reset_password_email(
         user_context,
     )
 
-    return CreateResetPasswordEmailOkResult()
+    return SendResetPasswordEmailEmailOkResult()

supertokens_python/recipe/thirdpartyemailpassword/interfaces.py

@@ -21,9 +21,9 @@
 CreateResetPasswordLinkUknownUserIdError = (
     EPInterfaces.CreateResetPasswordLinkUknownUserIdError
 )
-CreateResetPasswordEmailOkResult = EPInterfaces.CreateResetPasswordEmailOkResult
-CreateResetPasswordEmailUnknownUserIdError = (
-    EPInterfaces.CreateResetPasswordEmailUnknownUserIdError
+SendResetPasswordEmailEmailOkResult = EPInterfaces.SendResetPasswordEmailOkResult
+SendResetPasswordEmailUnknownUserIdError = (
+    EPInterfaces.SendResetPasswordEmailUnknownUserIdError
 )
 EmailPasswordEmailExistsGetOkResult = EPInterfaces.EmailExistsGetOkResult
 GeneratePasswordResetTokenPostOkResult = (