pan-scm-cli 1.3.2__tar.gz → 1.3.4__tar.gz

{pan_scm_cli-1.3.2 → pan_scm_cli-1.3.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pan-scm-cli
-Version: 1.3.2
+Version: 1.3.4
 Summary: CICD and Network Engineer-friendly CLI tool for Palo Alto Networks Strata Cloud Manager
 License-File: LICENSE
 Author: Calvin Remsburg

{pan_scm_cli-1.3.2 → pan_scm_cli-1.3.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "pan-scm-cli"
-version = "1.3.2"
+version = "1.3.4"
 description = "CICD and Network Engineer-friendly CLI tool for Palo Alto Networks Strata Cloud Manager"
 authors = ["Calvin Remsburg <dev@cdot.io>"]
 readme = "README.md"

{pan_scm_cli-1.3.2 → pan_scm_cli-1.3.4}/src/scm_cli/commands/incidents.py

@@ -5,6 +5,7 @@ from the SCM Unified Incident Framework.
 """
 
 import json
+from datetime import datetime, timezone
 
 import typer
 from rich.console import Console
@@ -29,6 +30,17 @@ PRODUCT_OPTION = typer.Option(None, "--product", "-p", help="Filter by product n
 JSON_OPTION = typer.Option(False, "--json", "-j", help="Output as JSON")
 
 
+def _format_epoch(epoch: int | str | None) -> str:
+    """Convert epoch timestamp (seconds or ms) to human-readable date."""
+    if epoch is None:
+        return ""
+    if isinstance(epoch, str):
+        return epoch
+    if epoch > 1_000_000_000_000:
+        return datetime.fromtimestamp(epoch / 1000, tz=timezone.utc).strftime("%Y-%m-%d %H:%M")
+    return datetime.fromtimestamp(epoch, tz=timezone.utc).strftime("%Y-%m-%d %H:%M")
+
+
 # =============================================================================================================================================================================================
 # INCIDENTS COMMANDS
 # =============================================================================================================================================================================================
@@ -64,13 +76,13 @@ def list_incidents(
             typer.echo("No incidents found")
             return
 
-        table = Table(title="Security Incidents")
+        table = Table(title="Security Incidents", show_lines=True)
         table.add_column("ID", style="cyan", no_wrap=True)
-        table.add_column("Status", style="white")
-        table.add_column("Severity", style="white")
-        table.add_column("Product", style="white")
-        table.add_column("Title", style="dim", max_width=40)
-        table.add_column("Raised", style="white")
+        table.add_column("Status", style="white", no_wrap=True)
+        table.add_column("Severity", style="white", no_wrap=True)
+        table.add_column("Product", style="white", no_wrap=True)
+        table.add_column("Title", style="dim", max_width=50)
+        table.add_column("Raised", style="white", no_wrap=True)
 
         severity_styles = {"critical": "red bold", "high": "red", "medium": "yellow", "low": "green", "informational": "dim"}
 
@@ -85,7 +97,7 @@ def list_incidents(
                 f"[{sev_style}]{sev}[/{sev_style}]",
                 str(inc.get("product", "")),
                 str(inc.get("title", "")),
-                str(inc.get("raised_time", "")),
+                _format_epoch(inc.get("raised_time")),
             )
 
         console.print(table)
@@ -122,8 +134,8 @@ def show_incident(
         typer.echo(f"Status:   {incident.get('status', '')}")
         typer.echo(f"Severity: {incident.get('severity', '')}")
         typer.echo(f"Product:  {incident.get('product', '')}")
-        typer.echo(f"Raised:   {incident.get('raised_time', '')}")
-        typer.echo(f"Updated:  {incident.get('updated_time', '')}")
+        typer.echo(f"Raised:   {_format_epoch(incident.get('raised_time'))}")
+        typer.echo(f"Updated:  {_format_epoch(incident.get('updated_time'))}")
         typer.echo(f"Title:    {incident.get('title', '')}")
 
         alerts = incident.get("alerts", [])
@@ -135,9 +147,23 @@ def show_incident(
                 state = alert.get("state", "")
                 typer.echo(f"  {i}. [{sev}] {title}   ({state})")
 
-        remediations = incident.get("remediations", "")
-        if remediations:
-            typer.echo(f"\nRemediations:\n  {remediations}")
+        remediations_raw = incident.get("remediations", "")
+        if remediations_raw:
+            typer.echo("\nRemediation:")
+            try:
+                parsed = json.loads(remediations_raw) if isinstance(remediations_raw, str) else remediations_raw
+                steps = parsed.get("remediations", []) if isinstance(parsed, dict) else []
+                for rem in steps:
+                    dc = rem.get("dynamic_content", {})
+                    for j, step in enumerate(dc.get("steps", []), 1):
+                        typer.echo(f"  {j}. {step.get('title', '').strip()}")
+                        desc = step.get("description", "").strip()
+                        if desc:
+                            typer.echo(f"     {desc}")
+                if not steps:
+                    typer.echo(f"  {remediations_raw}")
+            except (json.JSONDecodeError, AttributeError):
+                typer.echo(f"  {remediations_raw}")
 
         typer.echo()
 

{pan_scm_cli-1.3.2 → pan_scm_cli-1.3.4}/src/scm_cli/commands/local.py

@@ -67,6 +67,17 @@ def list_versions(
 
         console.print(table)
 
+    except ValueError as e:
+        if "Invalid error response format" in str(e):
+            typer.echo(
+                "Error: The Local Config API returned 404. "
+                "This API may not be available for your SCM tenant or device type. "
+                "Contact Palo Alto Networks support to verify Local Config API access.",
+                err=True,
+            )
+        else:
+            typer.echo(f"Error listing config versions: {e!s}", err=True)
+        raise typer.Exit(code=1) from e
     except Exception as e:
         typer.echo(f"Error listing config versions: {e!s}", err=True)
         raise typer.Exit(code=1) from e

{pan_scm_cli-1.3.2 → pan_scm_cli-1.3.4}/src/scm_cli/commands/operations.py

@@ -73,6 +73,17 @@ def _run_operation(device: str, operation: str, async_mode: bool, timeout: int)
 
         console.print(table)
 
+    except ValueError as e:
+        if "Invalid error response format" in str(e):
+            typer.echo(
+                f"Error: The Operations API returned 404 for {operation}. "
+                "This API may not be available for your SCM tenant or device type. "
+                "Contact Palo Alto Networks support to verify Operations API access.",
+                err=True,
+            )
+        else:
+            typer.echo(f"Error running {operation}: {e!s}", err=True)
+        raise typer.Exit(code=1) from e
     except Exception as e:
         typer.echo(f"Error running {operation}: {e!s}", err=True)
         raise typer.Exit(code=1) from e

{pan_scm_cli-1.3.2 → pan_scm_cli-1.3.4}/src/scm_cli/utils/sdk_client.py

@@ -16036,10 +16036,10 @@ class SCMClient:
     _SERIAL_PATTERN = __import__("re").compile(r"^\d{14,15}$")
 
     def resolve_device_serial(self, device: str) -> str:
-        """Resolve a device name or serial number to a serial number.
+        """Resolve a device name, hostname, or serial number to a serial number.
 
         Args:
-            device: Device name or serial number.
+            device: Device hostname, display name, or serial number.
 
         Returns:
             str: The 14-15 digit device serial number.
@@ -16057,12 +16057,14 @@ class SCMClient:
             return "007951000123456"
 
         try:
-            result = self.client.device.fetch(name=device)
-            if result is None:
-                raise ValueError(f"Device '{device}' not found in SCM")
-            serial = result.id
-            self.logger.info(f"Resolved '{device}' to serial {serial}")
-            return serial
+            all_devices = self.client.device.list()
+            search = device.lower()
+            for d in all_devices:
+                if any(search == (getattr(d, field, None) or "").lower() for field in ("hostname", "display_name", "name", "serial_number")):
+                    self.logger.info(f"Resolved '{device}' to serial {d.id}")
+                    return d.id
+            available = [f"  {d.hostname or d.display_name or d.name} ({d.id})" for d in all_devices]
+            raise ValueError(f"Device '{device}' not found. Available devices:\n" + "\n".join(available))
         except ValueError:
             raise
         except Exception as e:
@@ -16340,8 +16342,22 @@ class SCMClient:
             return self._MOCK_INCIDENTS[0]
 
         try:
-            result = self.client.incidents.get_details(incident_id=incident_id)
-            return json.loads(result.model_dump_json(exclude_unset=True))
+            # Bypass SDK's get_details() which has a parsing bug (passes whole
+            # response to model instead of extracting data[0] from the wrapper).
+            session = self.client.oauth_client.session
+            base = self.client.api_base_url
+            resp = session.get(
+                f"{base}/incidents/v1/details/{incident_id}",
+                headers={"X-PANW-Region": getattr(self.client, "_region", "americas")},
+            )
+            resp.raise_for_status()
+            body = resp.json()
+            items = body.get("data", [])
+            if not items:
+                raise ValueError(f"Incident {incident_id} not found")
+            return items[0]
+        except ValueError:
+            raise
         except Exception as e:
             self._handle_api_exception("fetching", "N/A", f"incident {incident_id}", e)