M kovalsky/userencoding (#546)

* fix user encoding

* date

Author: m-kovalsky

README.md

@@ -123,7 +123,7 @@ An even better way to ensure the semantic-link-labs library is available in your
 2. Select your newly created environment within the 'Environment' drop down in the navigation bar at the top of the notebook
 
 ## Version History
-* [0.9.5](https://github.com/microsoft/semantic-link-labs/releases/tag/0.9.5) (March 6, 2025)
+* [0.9.5](https://github.com/microsoft/semantic-link-labs/releases/tag/0.9.5) (March 7, 2025)
 * [0.9.4](https://github.com/microsoft/semantic-link-labs/releases/tag/0.9.4) (February 27, 2025)
 * [0.9.3](https://github.com/microsoft/semantic-link-labs/releases/tag/0.9.3) (February 13, 2025)
 * [0.9.2](https://github.com/microsoft/semantic-link-labs/releases/tag/0.9.2) (February 5, 2025)

src/sempy_labs/_helper_functions.py

@@ -30,6 +30,11 @@ def _build_url(url: str, params: dict) -> str:
     return url
 
 
+def _encode_user(user: str) -> str:
+
+    return urllib.parse.quote(user, safe="@")
+
+
 def create_abfss_path(
     lakehouse_id: UUID,
     lakehouse_workspace_id: UUID,

src/sempy_labs/admin/_workspaces.py

@@ -1,6 +1,7 @@
 from sempy_labs._helper_functions import (
     _base_api,
     _build_url,
+    _encode_user,
 )
 from uuid import UUID
 from typing import Optional
@@ -48,6 +49,8 @@ def add_user_to_workspace(
             f"{icons.red_dot} Invalid principal type. Please choose from {principal_types}"
         )
 
+    user = _encode_user(user)
+
     payload = {
         "identifier": user,  # identifier or emailAddress?
         "principalType": principal_type,
@@ -92,6 +95,7 @@ def delete_user_from_workspace(
 
     (workspace_name, workspace_id) = _resolve_workspace_name_and_id(workspace)
 
+    user = _encode_user(user)
     url = f"/v1.0/myorg/admin/groups/{workspace_id}/users/{user}"
 
     params = {}