python-netgear-switch-library 0.0.post336__tar.gz → 0.0.post337__tar.gz

{python_netgear_switch_library-0.0.post336 → python_netgear_switch_library-0.0.post337}/PKG-INFO

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

{python_netgear_switch_library-0.0.post336 → python_netgear_switch_library-0.0.post337}/src/netgear_switch/models.py

@@ -119,6 +119,38 @@ class MgmtIpConfig:
     base_mac: str | None = None
 
 
+@dataclass(frozen=True)
+class SyslogServer:
+    """One remote syslog collector the switch is configured to send to."""
+
+    host: str
+    port: int
+    #: Standard syslog severity, 0 (emergency) to 7 (debug). The switch sends
+    #: messages AT OR ABOVE this level. Cross-checked on m4300-24x: the SNMP
+    #: column reads 6 where ``show logging hosts`` prints "info".
+    severity: int
+    #: The switch's own word for the row's state, "Active" in the CLI table.
+    active: bool
+
+
+@dataclass(frozen=True)
+class SyslogConfig:
+    """Remote-logging configuration: whether it is on, and where it sends.
+
+    Deliberately narrower than everything ``show logging`` prints. The console
+    and buffered-logging columns are in the same vendor subtree, but only the
+    console pair could be decoded against captured CLI output; the buffered
+    severity did not match any column read, so it is left out rather than
+    guessed at. See ``VendorOids.syslog_*``.
+    """
+
+    enabled: bool
+    #: The source port the switch sends FROM (``Logging Client Local Port``),
+    #: not the collector's port -- that is per-server in ``servers``.
+    local_port: int
+    servers: tuple[SyslogServer, ...]
+
+
 @dataclass(frozen=True)
 class DetectedModel:
     """Result of identifying a switch's model over SNMP (sysObjectID + sysDescr).

{python_netgear_switch_library-0.0.post336 → python_netgear_switch_library-0.0.post337}/src/netgear_switch/protocols/snmp/oids.py

@@ -147,6 +147,32 @@ class VendorOids:
     """The ONE symbol every call site uses for the DHCP-mode OID. See
     DHCP_MODE_OID_SUFFIX above — UNVERIFIED, best-effort read only. No call site
     may hard-code a ``.99.1`` literal; they all reference this field."""
+    syslog_admin_mode: str
+    syslog_local_port: str
+    syslog_host_addr: str
+    syslog_host_port: str
+    syslog_host_severity: str
+    syslog_host_status: str
+    """Remote-logging configuration, under ``<base>.14`` on BOTH vendor
+    families -- 4526.10 (FASTPATH) and 4526.11 (S3300) share the column layout.
+
+    Located 2026-08-02 by reading each switch's own ``show logging`` /
+    ``show logging hosts`` and then searching a full walk for those values;
+    every field of the CLI output is accounted for by a column and the two
+    agree. On m4300-24x (10.1.5.13) the host row reads 10.1.5.1 / port 514 /
+    severity 6 / status 1 against a CLI table of ``10.1.5.1  info  514
+    Active`` -- so severity is the standard syslog scale (6 = info) and status
+    1 = Active.
+
+    ``<base>.17`` is NOT this: it looks like logging until you notice it holds
+    port 123 and the string "NTP Bits". It is SNTP, and this fleet's NTP server
+    and syslog server are the same host, which is what makes the confusion easy.
+
+    The admin-mode enum is ``1 = enabled, 2 = disabled``, confirmed twice over
+    on m4300-24x: syslog reads 1 while ``show logging`` says "Syslog Logging :
+    enabled", and the console column reads 2 while it says "Console Logging :
+    disabled". The console severity column independently reads 3 against a CLI
+    "error", matching the same syslog scale."""
     mgmt_write_addr_unverified: str
     mgmt_write_netmask_unverified: str
     mgmt_write_gateway_unverified: str
@@ -236,6 +262,12 @@ def vendor_oids(model: SwitchModel) -> VendorOids:
         box_psu_power=f"{base}.43.1.8.1.5",
         box_temp=f"{base}.43.1.15.1.3",
         dhcp_mode_unverified=f"{base}.{DHCP_MODE_OID_SUFFIX}",
+        syslog_admin_mode=f"{base}.14.1.4.1.0",
+        syslog_local_port=f"{base}.14.1.4.3.0",
+        syslog_host_addr=f"{base}.14.1.4.5.1.3",
+        syslog_host_port=f"{base}.14.1.4.5.1.4",
+        syslog_host_severity=f"{base}.14.1.4.5.1.5",
+        syslog_host_status=f"{base}.14.1.4.5.1.7",
         mgmt_write_addr_unverified=f"{base}.98.1",
         mgmt_write_netmask_unverified=f"{base}.98.2",
         mgmt_write_gateway_unverified=f"{base}.98.3",

{python_netgear_switch_library-0.0.post336 → python_netgear_switch_library-0.0.post337}/src/netgear_switch/protocols/snmp/parse.py

@@ -17,6 +17,8 @@ from ...models import (
     PortStats,
     PortStatus,
     Sensor,
+    SyslogConfig,
+    SyslogServer,
     VLANInfo,
 )
 from .client import SnmpError, SnmpRow
@@ -716,6 +718,70 @@ def _ipv4_from_rfc4293_index(rows: Sequence[SnmpRow]) -> str | None:
     return None
 
 
+#: The vendor admin-mode enum shared by every logging destination column.
+#: 1 = enabled, 2 = disabled -- confirmed twice on m4300-24x against its own
+#: ``show logging``: syslog reads 1 under "Syslog Logging : enabled" and the
+#: console column reads 2 under "Console Logging : disabled".
+_ADMIN_ENABLED = 1
+
+#: Row status in the syslog host table. 1 is what the CLI prints as "Active".
+_HOST_STATUS_ACTIVE = 1
+
+
+def _first_int(rows: Sequence[SnmpRow]) -> int | None:
+    """The value of a single-varbind scalar GET, when it is an integer."""
+    for row in rows:
+        if isinstance(row.value, int):
+            return row.value
+    return None
+
+
+def parse_syslog(
+    admin_mode: Sequence[SnmpRow],
+    local_port: Sequence[SnmpRow],
+    host_addr: Sequence[SnmpRow],
+    host_port: Sequence[SnmpRow],
+    host_severity: Sequence[SnmpRow],
+    host_status: Sequence[SnmpRow],
+    *,
+    addr_base: str,
+    port_base: str,
+    severity_base: str,
+    status_base: str,
+) -> SyslogConfig:
+    """Vendor logging columns -> ``SyslogConfig``.
+
+    The host table is indexed by an integer row id, and every per-host column is
+    matched to the address column by that index rather than by position -- so a
+    table with a gap in its indices (a deleted row) cannot silently shift one
+    row's port onto another row's address.
+
+    A row whose address is empty is skipped. The address is the only field that
+    makes a row meaningful, and reporting one collector fewer is far better than
+    inventing where logs are being sent.
+    """
+    addresses = index_str_column(host_addr, addr_base)
+    ports = index_int_column(host_port, port_base)
+    severities = index_int_column(host_severity, severity_base)
+    statuses = index_int_column(host_status, status_base)
+
+    servers = tuple(
+        SyslogServer(
+            host=address,
+            port=ports.get(index, 0),
+            severity=severities.get(index, 0),
+            active=statuses.get(index) == _HOST_STATUS_ACTIVE,
+        )
+        for index, address in sorted(addresses.items())
+        if address.strip()
+    )
+    return SyslogConfig(
+        enabled=_first_int(admin_mode) == _ADMIN_ENABLED,
+        local_port=_first_int(local_port) or 0,
+        servers=servers,
+    )
+
+
 def parse_mgmt_ip(
     addr: Sequence[SnmpRow],
     netmask: Sequence[SnmpRow],

{python_netgear_switch_library-0.0.post336 → python_netgear_switch_library-0.0.post337}/src/netgear_switch/snmp_read.py

@@ -23,6 +23,7 @@ if TYPE_CHECKING:
         PortStats,
         PortStatus,
         Sensor,
+        SyslogConfig,
         VLANInfo,
     )
     from .protocols.snmp.client import AsyncSnmpClient, SnmpClient
@@ -201,6 +202,36 @@ class SnmpReader:
         """
         return parse.parse_hostname(self.client.get([oids.SYS_NAME]))
 
+    def get_syslog(self) -> SyslogConfig:
+        """Remote-logging configuration: whether it is on, and where it sends.
+
+        VENDOR columns, so a model with no Netgear subtree cannot serve this.
+        ``gs728tpp`` is exactly that model -- a walk of ``1.3.6.1.4.1.4526``
+        answers ``noSuchObject`` -- and it is refused by name rather than
+        returned empty, which would read as "no collectors configured".
+        """
+        if not oids.has_vendor_oids(self.model):
+            raise UnsupportedCapabilityError(
+                f"model {self.model.key!r} registers no Netgear vendor OID "
+                "subtree, and the logging columns are vendor-only; an empty "
+                "result here would be indistinguishable from a switch with no "
+                "syslog collectors configured"
+            )
+        vo = oids.vendor_oids(self.model)
+        w = self.client.walk
+        return parse.parse_syslog(
+            self.client.get([vo.syslog_admin_mode]),
+            self.client.get([vo.syslog_local_port]),
+            w(vo.syslog_host_addr),
+            w(vo.syslog_host_port),
+            w(vo.syslog_host_severity),
+            w(vo.syslog_host_status),
+            addr_base=vo.syslog_host_addr,
+            port_base=vo.syslog_host_port,
+            severity_base=vo.syslog_host_severity,
+            status_base=vo.syslog_host_status,
+        )
+
     def get_system_info(self) -> DetectedModel:
         """Identify this switch's model via sysDescr (see ``read_system_info``).