bt-cli 0.4.33__tar.gz → 0.4.34__tar.gz

{bt_cli-0.4.33 → bt_cli-0.4.34}/CLAUDE.md

@@ -1,6 +1,6 @@
 # BT-CLI
 
-BeyondTrust Platform CLI for Password Safe, Entitle, PRA, EPM Windows, and EPM Linux. **Version: 0.4.33**
+BeyondTrust Platform CLI for Password Safe, Entitle, PRA, EPM Windows, and EPM Linux. **Version: 0.4.34**
 
 ## Setup
 

{bt_cli-0.4.33 → bt_cli-0.4.34}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: bt-cli
-Version: 0.4.33
+Version: 0.4.34
 Summary: BeyondTrust Platform CLI (unofficial) - Password Safe, Entitle, PRA, EPM
 Author-email: Dave Grendysz <dgrendysz@beyondtrust.com>
 License: MIT

{bt_cli-0.4.33 → bt_cli-0.4.34}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "bt-cli"
-version = "0.4.33"
+version = "0.4.34"
 description = "BeyondTrust Platform CLI (unofficial) - Password Safe, Entitle, PRA, EPM"
 readme = "README.md"
 requires-python = ">=3.9"

{bt_cli-0.4.33 → bt_cli-0.4.34}/src/bt_cli/__init__.py

@@ -1,3 +1,3 @@
 """BeyondTrust Unified Admin CLI."""
 
-__version__ = "0.4.33"
+__version__ = "0.4.34"

{bt_cli-0.4.33 → bt_cli-0.4.34}/src/bt_cli/data/CLAUDE.md

@@ -1,6 +1,6 @@
 # BT-CLI
 
-BeyondTrust Platform CLI for Password Safe, Entitle, PRA, EPM Windows, and EPM Linux. **Version: 0.4.33**
+BeyondTrust Platform CLI for Password Safe, Entitle, PRA, EPM Windows, and EPM Linux. **Version: 0.4.34**
 
 ## Setup
 

{bt_cli-0.4.33 → bt_cli-0.4.34}/src/bt_cli/epml/client/base.py

@@ -313,10 +313,10 @@ class EPMLClient:
         h = self.host(host_id)
         return self.post(f"/api/pbul/{h}/rbp/cmdgrps", json=data)
 
-    def delete_cmdgrps(self, ids: List[int], host_id: Optional[int] = None) -> Any:
+    def delete_cmdgrps(self, ids: List[int], host_id: Optional[int] = None) -> None:
         h = self.host(host_id)
-        # /api/pbul/{hostid}/rbp/cmdgrps DELETE deletes a list of IDs in body.
-        return self._delete_with_body(f"/api/pbul/{h}/rbp/cmdgrps", ids)
+        for cid in ids:
+            self._delete_with_query(f"/api/pbul/{h}/rbp/cmdgrps", {"id": cid})
 
     def list_cmdgrp_commands(self, cmdgrp_id: int, host_id: Optional[int] = None) -> Any:
         h = self.host(host_id)
@@ -338,11 +338,15 @@ class EPMLClient:
 
     def create_hostgrp(self, data: Dict[str, Any], host_id: Optional[int] = None) -> Any:
         h = self.host(host_id)
-        return self.post(f"/api/pbul/{h}/rbp/hostgrps", json=data)
+        # Server requires `type` ∈ {"I","E"}. Default to Internal if omitted.
+        body = dict(data)
+        body.setdefault("type", "I")
+        return self.post(f"/api/pbul/{h}/rbp/hostgrps", json=body)
 
-    def delete_hostgrps(self, ids: List[int], host_id: Optional[int] = None) -> Any:
+    def delete_hostgrps(self, ids: List[int], host_id: Optional[int] = None) -> None:
         h = self.host(host_id)
-        return self._delete_with_body(f"/api/pbul/{h}/rbp/hostgrps", ids)
+        for hid in ids:
+            self._delete_with_query(f"/api/pbul/{h}/rbp/hostgrps", {"id": hid})
 
     def list_hostgrp_hosts(self, hostgrp_id: int, host_id: Optional[int] = None) -> Any:
         h = self.host(host_id)
@@ -364,11 +368,15 @@ class EPMLClient:
 
     def create_usergrp(self, data: Dict[str, Any], host_id: Optional[int] = None) -> Any:
         h = self.host(host_id)
-        return self.post(f"/api/pbul/{h}/rbp/usergrps", json=data)
+        # Server requires `type` ∈ {"I","E"}. Default to Internal if omitted.
+        body = dict(data)
+        body.setdefault("type", "I")
+        return self.post(f"/api/pbul/{h}/rbp/usergrps", json=body)
 
-    def delete_usergrps(self, ids: List[int], host_id: Optional[int] = None) -> Any:
+    def delete_usergrps(self, ids: List[int], host_id: Optional[int] = None) -> None:
         h = self.host(host_id)
-        return self._delete_with_body(f"/api/pbul/{h}/rbp/usergrps", ids)
+        for uid in ids:
+            self._delete_with_query(f"/api/pbul/{h}/rbp/usergrps", {"id": uid})
 
     def list_usergrp_users(self, usergrp_id: int, host_id: Optional[int] = None) -> Any:
         h = self.host(host_id)
@@ -392,9 +400,10 @@ class EPMLClient:
         h = self.host(host_id)
         return self.post(f"/api/pbul/{h}/rbp/tmdategrps", json=data)
 
-    def delete_tmdategrps(self, ids: List[int], host_id: Optional[int] = None) -> Any:
+    def delete_tmdategrps(self, ids: List[int], host_id: Optional[int] = None) -> None:
         h = self.host(host_id)
-        return self._delete_with_body(f"/api/pbul/{h}/rbp/tmdategrps", ids)
+        for tid in ids:
+            self._delete_with_query(f"/api/pbul/{h}/rbp/tmdategrps", {"id": tid})
 
     # ----- RBP: roles ---------------------------------------------------
 
@@ -404,11 +413,15 @@ class EPMLClient:
 
     def create_role(self, data: Dict[str, Any], host_id: Optional[int] = None) -> Any:
         h = self.host(host_id)
-        return self.post(f"/api/pbul/{h}/rbp/roles", json=data)
+        # Server requires `action` ∈ {"A","R"} (Allow / Reject). Default Allow.
+        body = dict(data)
+        body.setdefault("action", "A")
+        return self.post(f"/api/pbul/{h}/rbp/roles", json=body)
 
-    def delete_roles(self, ids: List[int], host_id: Optional[int] = None) -> Any:
+    def delete_roles(self, ids: List[int], host_id: Optional[int] = None) -> None:
         h = self.host(host_id)
-        return self._delete_with_body(f"/api/pbul/{h}/rbp/roles", ids)
+        for rid in ids:
+            self._delete_with_query(f"/api/pbul/{h}/rbp/roles", {"id": rid})
 
     def duplicate_role(self, role_id: int, host_id: Optional[int] = None) -> Any:
         h = self.host(host_id)
@@ -569,6 +582,20 @@ class EPMLClient:
         except ValueError:
             return response.text
 
+    def _delete_with_query(self, spec_path: str, params: Dict[str, Any]) -> None:
+        """DELETE with query parameters (no body).
+
+        RBP collection deletes (`/pbul/{hostid}/rbp/{entity}`) take `id` as a
+        query param, not a JSON body. Repeated `?id=A&id=B` is accepted by the
+        server but only the first value is honored — callers should loop.
+        """
+        response = self._client.delete(
+            self._build_url(spec_path),
+            headers=self._headers(),
+            params=params,
+        )
+        response.raise_for_status()
+
     # ----- iologs --------------------------------------------------------
 
     def list_iologs(

{bt_cli-0.4.33 → bt_cli-0.4.34}/src/bt_cli/epml/commands/rbp_hostgrps.py

@@ -45,13 +45,17 @@ def list_hostgrps(
 def create_hostgrp(
     name: str = typer.Option(..., "--name", "-n"),
     description: str = typer.Option("", "--description", "-d"),
+    type_: str = typer.Option("I", "--type", "-t", help="Group type: I=Internal, E=External"),
     host: Optional[int] = _host_opt(),
 ):
-    """Create a host group."""
+    """Create a host group. API requires `type` ∈ {I,E}; defaults to Internal."""
     from bt_cli.epml.client import get_client
+    if type_ not in ("I", "E"):
+        typer.echo(f"--type must be 'I' or 'E', got {type_!r}", err=True)
+        raise typer.Exit(2)
     try:
         with get_client() as c:
-            result = c.create_hostgrp({"name": name, "description": description}, host_id=host)
+            result = c.create_hostgrp({"name": name, "description": description, "type": type_}, host_id=host)
         print_json(result)
     except httpx.HTTPStatusError as e:
         print_api_error(e, "create hostgrp"); raise typer.Exit(1)

{bt_cli-0.4.33 → bt_cli-0.4.34}/src/bt_cli/epml/commands/rbp_roles.py

@@ -45,13 +45,21 @@ def list_roles(
 def create_role(
     name: str = typer.Option(..., "--name", "-n"),
     description: str = typer.Option("", "--description", "-d"),
+    action: str = typer.Option("A", "--action", "-a", help="Role verdict: A=Allow, R=Reject"),
     host: Optional[int] = _host_opt(),
 ):
-    """Create a role."""
+    """Create a role.
+
+    The API requires `action` to be exactly `A` (Allow) or `R` (Reject) —
+    not 'Allow'/'Reject'. Defaults to A.
+    """
     from bt_cli.epml.client import get_client
+    if action not in ("A", "R"):
+        typer.echo(f"--action must be 'A' or 'R', got {action!r}", err=True)
+        raise typer.Exit(2)
     try:
         with get_client() as c:
-            result = c.create_role({"name": name, "description": description}, host_id=host)
+            result = c.create_role({"name": name, "description": description, "action": action}, host_id=host)
         print_json(result)
     except httpx.HTTPStatusError as e:
         print_api_error(e, "create role"); raise typer.Exit(1)

{bt_cli-0.4.33 → bt_cli-0.4.34}/src/bt_cli/epml/commands/rbp_usergrps.py

@@ -45,13 +45,17 @@ def list_usergrps(
 def create_usergrp(
     name: str = typer.Option(..., "--name", "-n"),
     description: str = typer.Option("", "--description", "-d"),
+    type_: str = typer.Option("I", "--type", "-t", help="Group type: I=Internal, E=External"),
     host: Optional[int] = _host_opt(),
 ):
-    """Create a user group."""
+    """Create a user group. API requires `type` ∈ {I,E}; defaults to Internal."""
     from bt_cli.epml.client import get_client
+    if type_ not in ("I", "E"):
+        typer.echo(f"--type must be 'I' or 'E', got {type_!r}", err=True)
+        raise typer.Exit(2)
     try:
         with get_client() as c:
-            result = c.create_usergrp({"name": name, "description": description}, host_id=host)
+            result = c.create_usergrp({"name": name, "description": description, "type": type_}, host_id=host)
         print_json(result)
     except httpx.HTTPStatusError as e:
         print_api_error(e, "create usergrp"); raise typer.Exit(1)

{bt_cli-0.4.33 → bt_cli-0.4.34}/tests/epml/test_client.py

@@ -146,15 +146,18 @@ class TestHTTPVerbs:
         assert c.delete_testsuite(1) is None
 
     @respx.mock
-    def test_delete_with_body(self, epml_config):
-        """DELETE with a JSON body for batch deletes."""
+    def test_delete_via_query_param_one_per_id(self, epml_config):
+        """RBP collection deletes use ?id=N query string, single ID per call."""
         route = respx.delete(
             "https://mock-gateway.example.com/site/SITE/epm/linux/pbul/100/rbp/cmdgrps"
-        ).mock(return_value=httpx.Response(204))
+        ).mock(return_value=httpx.Response(200))
 
         c = EPMLClient(epml_config)
         c.delete_cmdgrps([1, 2, 3])
-        assert route.calls.last.request.read() == b"[1,2,3]"
+        # Three separate requests, one per id
+        assert route.call_count == 3
+        sent_ids = sorted(call.request.url.params["id"] for call in route.calls)
+        assert sent_ids == ["1", "2", "3"]
 
 
 # ---------------------------------------------------------------------------