Merge pull request #476 from supertokens/fix/domain-normalisation

fix: dashboard connection uri path

Author: rishabhpoddar

CHANGELOG.md

@@ -8,6 +8,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [unreleased]
 
+## [0.18.8] - 2024-02-29
+
+- Fixes dashboard URI path. Now it returns the complete user given path instead of just the normalized connectionURI domain.
+
 ## [0.18.7] - 2024-01-17
 
 - Fixes `connection_uri` normalisation in the dashboard recipe.

setup.py

@@ -70,7 +70,7 @@
 
 setup(
     name="supertokens_python",
-    version="0.18.7",
+    version="0.18.8",
     author="SuperTokens",
     license="Apache 2.0",
     author_email="[email protected]",

supertokens_python/constants.py

@@ -14,7 +14,7 @@
 from __future__ import annotations
 
 SUPPORTED_CDI_VERSIONS = ["3.0"]
-VERSION = "0.18.7"
+VERSION = "0.18.8"
 TELEMETRY = "/telemetry"
 USER_COUNT = "/users/count"
 USER_DELETE = "/user/remove"

supertokens_python/recipe/dashboard/api/implementation.py

@@ -50,9 +50,18 @@ async def dashboard_get(
             connection_uri = ""
             super_tokens_instance = Supertokens.get_instance()
             auth_mode = options.config.auth_mode
-            connection_uri = NormalisedURLDomain(
-                super_tokens_instance.supertokens_config.connection_uri.split(";")[0]
-            ).get_as_string_dangerous()
+            connection_uri = (
+                NormalisedURLDomain(
+                    super_tokens_instance.supertokens_config.connection_uri.split(";")[
+                        0
+                    ]
+                ).get_as_string_dangerous()
+                + NormalisedURLPath(
+                    super_tokens_instance.supertokens_config.connection_uri.split(";")[
+                        0
+                    ]
+                ).get_as_string_dangerous()
+            )
 
             dashboard_path = options.app_info.api_base_path.append(
                 NormalisedURLPath(DASHBOARD_API)

tests/dashboard/test_dashboard.py

@@ -163,7 +163,7 @@ async def test_connection_uri_has_http_prefix_if_localhost(app: TestClient):
 
 async def test_connection_uri_has_https_prefix_if_not_localhost(app: TestClient):
     start_st()
-    connection_uri = "https://try.supertokens.com"
+    connection_uri = "https://try.supertokens.com/appid-public"
     connection_uri_without_protocol = connection_uri.replace("https://", "")
 
     st_args = get_st_init_args(
@@ -189,8 +189,9 @@ async def test_connection_uri_has_https_prefix_if_not_localhost(app: TestClient)
 async def test_that_first_connection_uri_is_selected_among_multiple_uris(
     app: TestClient,
 ):
-    first_connection_uri = start_st()
-    second_connection_uri = "https://try.supertokens.com"
+    start_st()
+    first_connection_uri = "https://try.supertokens.com/appid-public"
+    second_connection_uri = "https://test.supertokens.com/"
     multiple_connection_uris = f"{first_connection_uri};{second_connection_uri}"
 
     st_args = get_st_init_args(