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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pan-scm-cli
-Version: 1.3.2
+Version: 1.3.3
 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.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "pan-scm-cli"
-version = "1.3.2"
+version = "1.3.3"
 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.3}/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.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 = Table(title="Security Incidents", show_lines=True)
+        table.add_column("ID", style="cyan", no_wrap=True, max_width=12)
+        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"}
 
@@ -79,13 +91,15 @@ def list_incidents(
             sev_style = severity_styles.get(sev, "white")
             status_val = inc.get("status", "")
             status_style = "green" if status_val == "closed" else ("yellow" if status_val == "in_progress" else "white")
+            inc_id = str(inc.get("incident_id", ""))
+            short_id = inc_id[:8] + "..." if len(inc_id) > 12 else inc_id
             table.add_row(
-                str(inc.get("incident_id", "")),
+                short_id,
                 f"[{status_style}]{status_val}[/{status_style}]",
                 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 +136,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", [])

{pan_scm_cli-1.3.2 → pan_scm_cli-1.3.3}/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.3}/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.3}/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: