python-netgear-switch-library 0.0.post338__tar.gz → 0.0.post340__tar.gz

{python_netgear_switch_library-0.0.post338 → python_netgear_switch_library-0.0.post340}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-netgear-switch-library
-Version: 0.0.post338
+Version: 0.0.post340
 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.post338 → python_netgear_switch_library-0.0.post340}/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.post338'
-__version_tuple__ = version_tuple = (0, 0, 'post338')
+__version__ = version = '0.0.post340'
+__version_tuple__ = version_tuple = (0, 0, 'post340')
 
 __commit_id__ = commit_id = None

{python_netgear_switch_library-0.0.post338 → python_netgear_switch_library-0.0.post340}/src/netgear_switch/aio_api.py

@@ -447,6 +447,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_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)
+
     async def nsdp_device(self) -> NsdpDevice:
         """Async twin of ``SyncSwitch.nsdp_device`` -- see there."""
         reader = self._reader_for(Backend.NSDP)

{python_netgear_switch_library-0.0.post338 → python_netgear_switch_library-0.0.post340}/src/netgear_switch/capabilities.py

@@ -113,6 +113,7 @@ READ_OPERATIONS: tuple[Operation, ...] = (
     Operation("get_poe", OperationKind.READ, "Per-port PoE status and power draw"),
     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(
         "nsdp_device",
         OperationKind.READ,
@@ -247,7 +248,7 @@ def _http_path_for(spec: HttpModelSpec, op: Operation) -> str | None:
     ops with composite conditions defer to the reader's own helpers so there is
     exactly one definition of "this UI can answer that".
     """
-    from .http_read import _mgmt_ip_path, _supports_sensors
+    from .http_read import _has_sysinfo_hostname, _mgmt_ip_path, _supports_sensors
 
     simple: dict[str, str | None] = {
         "get_ports": spec.dashboard_path,
@@ -269,6 +270,11 @@ def _http_path_for(spec: HttpModelSpec, op: Operation) -> str | None:
     }
     if op.name == "get_sensors":
         return spec.sysinfo_path if _supports_sensors(spec) else None
+    if op.name == "get_hostname":
+        # Only two identity pages carry the field (gs110emx's sysInfo.html and
+        # gs105pe's switch_info.cgi); the reader's own predicate decides, so
+        # there is one definition rather than two that can drift.
+        return spec.sysinfo_path if _has_sysinfo_hostname(spec) else None
     if op.name == "get_mgmt_ip":
         return _mgmt_ip_path(spec)
     if op.name == "set_mgmt_ip":

{python_netgear_switch_library-0.0.post338 → python_netgear_switch_library-0.0.post340}/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.post338 → python_netgear_switch_library-0.0.post340}/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.post338 → python_netgear_switch_library-0.0.post340}/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.post338 → python_netgear_switch_library-0.0.post340}/src/netgear_switch/sync_api.py

@@ -552,6 +552,20 @@ class SyncSwitch:
     def get_mgmt_ip(self, *, backend: Backend | None = None) -> MgmtIpConfig:
         return self._read(lambda r: r.get_mgmt_ip(), backend)
 
+    def get_hostname(self, *, backend: Backend | None = None) -> str:
+        """The switch's host name.
+
+        Every backend can answer, but from a different place: SNMP reads the
+        standard ``sysName`` scalar, NSDP the ``HOSTNAME`` tag, the CLI
+        ``show hosts``, and the web UI the device-identity page -- on the two
+        dialects whose page carries the field; the rest refuse by name.
+
+        The CLI deliberately parses ``show hosts`` rather than
+        ``show running-config``: the two report different values on real
+        hardware, and only ``show hosts`` agrees with ``sysName``.
+        """
+        return self._read(lambda r: r.get_hostname(), backend)
+
     def nsdp_device(self) -> NsdpDevice:
         """Return the COMPLETE raw ``NsdpDevice`` for this switch: model, MAC,
         hostname, mgmt IP, firmware, DHCP mode, port count, serial number,