python-netgear-switch-library 0.0.post339__tar.gz → 0.0.post341__tar.gz

{python_netgear_switch_library-0.0.post339 → python_netgear_switch_library-0.0.post341}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-netgear-switch-library
-Version: 0.0.post339
+Version: 0.0.post341
 Summary: Python library and CLI to query and control Netgear switches over SNMP, NSDP and HTTP.
 Author-email: Tim Ansell <me@mith.ro>
 License-Expression: Apache-2.0

{python_netgear_switch_library-0.0.post339 → python_netgear_switch_library-0.0.post341}/src/netgear_switch/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.0.post339'
-__version_tuple__ = version_tuple = (0, 0, 'post339')
+__version__ = version = '0.0.post341'
+__version_tuple__ = version_tuple = (0, 0, 'post341')
 
 __commit_id__ = commit_id = None

{python_netgear_switch_library-0.0.post339 → python_netgear_switch_library-0.0.post341}/src/netgear_switch/aio_api.py

@@ -95,6 +95,7 @@ if TYPE_CHECKING:
         PortStats,
         PortStatus,
         Sensor,
+        SyslogConfig,
         VLANInfo,
         VlanMode,
     )
@@ -447,6 +448,10 @@ class AsyncSwitch:
     async def get_mgmt_ip(self, *, backend: Backend | None = None) -> MgmtIpConfig:
         return await self._read(lambda r: r.get_mgmt_ip(), backend)
 
+    async def get_syslog(self, *, backend: Backend | None = None) -> SyslogConfig:
+        """Async twin of ``SyncSwitch.get_syslog`` -- see there."""
+        return await self._read(lambda r: r.get_syslog(), backend)
+
     async def get_hostname(self, *, backend: Backend | None = None) -> str:
         """Async twin of ``SyncSwitch.get_hostname`` -- see there."""
         return await self._read(lambda r: r.get_hostname(), backend)

{python_netgear_switch_library-0.0.post339 → python_netgear_switch_library-0.0.post341}/src/netgear_switch/capabilities.py

@@ -114,6 +114,19 @@ READ_OPERATIONS: tuple[Operation, ...] = (
     Operation("get_sensors", OperationKind.READ, "Fan/PSU/temperature sensors"),
     Operation("get_mgmt_ip", OperationKind.READ, "Management IP configuration"),
     Operation("get_hostname", OperationKind.READ, "The switch's host name"),
+    Operation(
+        "get_syslog",
+        OperationKind.READ,
+        "Remote-logging configuration and collectors",
+        # SNMP and the CLI are the two backends that READ this today, and the
+        # two agree field-for-field on live hardware. Restricting the op here
+        # states what the LIBRARY serves; it deliberately does NOT assert that
+        # a web UI cannot show syslog. No dialect's syslog page has been
+        # located or captured yet, so HTTP is an unbuilt implementation -- a
+        # bug to fix, in this project's terms, not a device limitation. NSDP
+        # has no logging tag in the exhaustive tag sweep of a live GS110EMX.
+        backends=frozenset({Backend.SNMP}) | _CLI_BACKENDS,
+    ),
     Operation(
         "nsdp_device",
         OperationKind.READ,
@@ -213,6 +226,17 @@ def _snmp_support(model: SwitchModel, op: Operation) -> tuple[Support, str]:
             f"model {model.key!r} registers no Netgear vendor OID subtree, and "
             "the management-IP write columns are vendor-only",
         )
+    if op.name == "get_syslog" and not oids.has_vendor_oids(model):
+        # Logging lives at <vendor base>.14 on both vendor families, so a model
+        # whose agent registers no 4526 subtree at all (gs728tpp -- a walk of
+        # 1.3.6.1.4.1.4526 answers noSuchObject) has nothing to read.
+        # SnmpReader.get_syslog refuses by name for the same reason: an empty
+        # result would be indistinguishable from a switch with no collectors.
+        return (
+            Support.UNSUPPORTED,
+            f"model {model.key!r} registers no Netgear vendor OID subtree, and "
+            "the logging columns are vendor-only",
+        )
     if op.name == "get_macs" and not model.has_mac_table:  # pragma: no cover
         # Unreachable today: has_mac_table IS "has an SNMP backend". Kept so the
         # rule tracks the property rather than assuming its current definition.

{python_netgear_switch_library-0.0.post339 → python_netgear_switch_library-0.0.post341}/src/netgear_switch/cli_read.py

@@ -36,6 +36,7 @@ if TYPE_CHECKING:
         PortStats,
         PortStatus,
         Sensor,
+        SyslogConfig,
         VLANInfo,
     )
     from .registry import SwitchModel
@@ -103,6 +104,18 @@ class CliReader:
     def get_mgmt_ip(self) -> MgmtIpConfig:
         return parse.parse_mgmt_ip(self.session.run(self._spec.network_cmd))
 
+    def get_syslog(self) -> SyslogConfig:
+        """Remote-logging configuration, from ``show logging`` + its host table.
+
+        Two commands because the switch splits it that way: the globals live in
+        ``show logging`` and the collectors in ``show logging hosts``. The host
+        table's column set differs by firmware -- see ``parse.parse_syslog``.
+        """
+        return parse.parse_syslog(
+            self.session.run(self._spec.logging_cmd),
+            self.session.run(self._spec.logging_hosts_cmd),
+        )
+
     def get_hostname(self) -> str:
         """The switch's host name, from ``show hosts``.
 

{python_netgear_switch_library-0.0.post339 → python_netgear_switch_library-0.0.post341}/src/netgear_switch/protocols/cli/commands.py

@@ -103,6 +103,10 @@ class CliModelSpec:
     # SNMP's sysName, so it is what keeps the two backends returning the same
     # answer for the same switch.
     hosts_cmd: str = "show hosts"
+    # Remote logging. `show syslog` does NOT exist -- all three FASTPATH
+    # switches answered "% Invalid input detected" on 2026-08-02.
+    logging_cmd: str = "show logging"
+    logging_hosts_cmd: str = "show logging hosts"
     # Global-config directive. Quoted on the wire by the device's own
     # running-config output ('hostname "sw-netgear-m4300-24x"').
     hostname_config_cmd: str = "hostname {name}"

{python_netgear_switch_library-0.0.post339 → python_netgear_switch_library-0.0.post341}/src/netgear_switch/protocols/cli/parse.py

@@ -42,6 +42,8 @@ from ...models import (
     PortStats,
     PortStatus,
     Sensor,
+    SyslogConfig,
+    SyslogServer,
     VLANInfo,
 )
 
@@ -606,6 +608,87 @@ def parse_environment(text: str) -> list[Sensor]:
     return out
 
 
+# ---------------------------------------------------------------------------
+# show logging / show logging hosts -> syslog configuration
+# ---------------------------------------------------------------------------
+
+#: Syslog severity names as FASTPATH prints them, to the standard numbers the
+#: SNMP columns carry. Cross-checked on m4300-24x: `show logging hosts` prints
+#: "info" where the SNMP severity column reads 6.
+_SEVERITY_NAMES = {
+    "emergency": 0,
+    "alert": 1,
+    "critical": 2,
+    "error": 3,
+    "warning": 4,
+    "notice": 5,
+    "info": 6,
+    "informational": 6,
+    "debug": 7,
+}
+
+
+def _colon_fields(text: str) -> dict[str, str]:
+    """Parse ``Label   : value`` lines, which `show logging` uses.
+
+    Distinct from ``labelled_values`` above: that reads the dotted-leader form
+    (``Label........ value``) which `show hosts` and `show network` use. The
+    logging command uses a colon instead, so it needs its own reader.
+    """
+    out: dict[str, str] = {}
+    for line in text.splitlines():
+        if ":" not in line:
+            continue
+        label, _, value = line.partition(":")
+        label = label.strip()
+        if label:
+            out[label] = value.strip()
+    return out
+
+
+def parse_syslog(logging_text: str, hosts_text: str) -> SyslogConfig:
+    """``show logging`` + ``show logging hosts`` -> ``SyslogConfig``.
+
+    Captured 2026-08-02 from m4300-24x (10.1.5.13), m4300-16x (10.1.5.20) and
+    gsm7252ps (10.1.5.22)::
+
+        Syslog Logging                      : enabled
+        Logging Client Local Port           : 514
+
+        Index   IP Address/Hostname     Severity    Port   Status  Mode  Auth  Cert#
+        ----- ------------------------ ---------- ------ --------- ----- ----- -----
+        1     10.1.5.1                 info       514    Active    udp
+
+    **The host table's column set differs by firmware.** The M4300s emit eight
+    columns (through ``Cert#``); the gsm7252ps emits only the first five. Both
+    are parsed by taking the first five whitespace-separated fields and ignoring
+    anything after ``Status``, so neither shape can shift a value into the wrong
+    field -- the same class of trap as the VLAN PortList width.
+    """
+    fields = _colon_fields(logging_text)
+    enabled = fields.get("Syslog Logging", "").strip().lower() == "enabled"
+    port_text = fields.get("Logging Client Local Port", "").strip()
+    local_port = int(port_text) if port_text.isdigit() else 0
+
+    servers: list[SyslogServer] = []
+    for line in hosts_text.splitlines():
+        cells = line.split()
+        # A data row starts with the integer index; the header and the dashed
+        # rule under it do not, which is what filters them out.
+        if len(cells) < 5 or not cells[0].isdigit():
+            continue
+        _index, host, severity, port, status = cells[:5]
+        servers.append(
+            SyslogServer(
+                host=host,
+                port=int(port) if port.isdigit() else 0,
+                severity=_SEVERITY_NAMES.get(severity.lower(), 0),
+                active=status.lower() == "active",
+            )
+        )
+    return SyslogConfig(enabled=enabled, local_port=local_port, servers=tuple(servers))
+
+
 # ---------------------------------------------------------------------------
 # show hosts -> host name
 # ---------------------------------------------------------------------------

{python_netgear_switch_library-0.0.post339 → python_netgear_switch_library-0.0.post341}/src/netgear_switch/sync_api.py

@@ -157,6 +157,7 @@ if TYPE_CHECKING:
         PortStats,
         PortStatus,
         Sensor,
+        SyslogConfig,
         VLANInfo,
         VlanMode,
     )
@@ -552,6 +553,17 @@ class SyncSwitch:
     def get_mgmt_ip(self, *, backend: Backend | None = None) -> MgmtIpConfig:
         return self._read(lambda r: r.get_mgmt_ip(), backend)
 
+    def get_syslog(self, *, backend: Backend | None = None) -> SyslogConfig:
+        """Remote-logging configuration: whether it is on, and its collectors.
+
+        Served over SNMP and the FASTPATH CLI, which agree field-for-field on
+        live hardware. ``gs728tpp`` is refused over SNMP by name: it registers
+        no Netgear vendor subtree, and the logging columns are vendor-only, so
+        an empty result would be indistinguishable from a switch with no
+        collectors configured.
+        """
+        return self._read(lambda r: r.get_syslog(), backend)
+
     def get_hostname(self, *, backend: Backend | None = None) -> str:
         """The switch's host name.