python-netgear-switch-library 0.0.post346__tar.gz → 0.0.post348__tar.gz

{python_netgear_switch_library-0.0.post346 → python_netgear_switch_library-0.0.post348}/PKG-INFO

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

{python_netgear_switch_library-0.0.post346 → python_netgear_switch_library-0.0.post348}/src/netgear_switch/aio_api.py

@@ -661,6 +661,21 @@ class AsyncSwitch:
             lambda w: w.clear_poe_fault(port, force=force, timeouts=timeouts), backend
         )
 
+    async def set_syslog_enabled(
+        self,
+        enabled: bool,
+        *,
+        force: bool = False,
+        backend: Backend | None = None,
+    ) -> None:
+        """Turn remote syslog on or off.
+
+        Served over SNMP. Adding or removing a COLLECTOR is a separate,
+        unbuilt operation -- it needs a row-status write that has not
+        been driven against hardware.
+        """
+        await self._write(lambda w: w.set_syslog_enabled(enabled, force=force), backend)
+
     async def set_hostname(
         self, name: str, *, force: bool = False, backend: Backend | None = None
     ) -> None:

{python_netgear_switch_library-0.0.post346 → python_netgear_switch_library-0.0.post348}/src/netgear_switch/capabilities.py

@@ -175,16 +175,21 @@ WRITE_OPERATIONS: tuple[Operation, ...] = (
         "set_hostname",
         OperationKind.WRITE,
         "Set the switch's host name",
-        # SNMP (sysName, confirmed writable on all five reachable switches) and
-        # the FASTPATH CLI (`hostname`) are what the library writes today.
+        # SNMP (sysName), NSDP (tag 0x0003) and the FASTPATH CLI (`hostname`),
+        # each confirmed writable against real hardware:
         #
-        # This restriction makes a REAL GAP visible rather than hiding it: the
-        # Plus models have neither backend, so this library currently cannot
-        # rename a gs110emx, gs105pe or gs305ep at all. Both routes exist on the
-        # devices -- NSDP carries a writable HOSTNAME tag and two web UIs expose
-        # a switch_name field -- so this is unbuilt work, not a device limit,
-        # and the support table should show the hole until it is filled.
-        backends=frozenset({Backend.SNMP}) | _CLI_BACKENDS,
+        #   SNMP  all five reachable switches accepted a SET of the value they
+        #         already held -- a zero-impact writability probe
+        #   NSDP  live round trip on gs110emx3 (10.1.5.27): recorded the prior
+        #         name, wrote a throwaway, read it back, restored, re-read
+        #   CLI   round-trips on all four CLI models
+        #
+        # That closes the Plus-model hole this restriction used to expose: every
+        # registered model now has at least one backend that can rename it. HTTP
+        # is still absent -- two web UIs carry a switch_name field but the write
+        # is unbuilt -- so the restriction stays, and states what the library
+        # serves rather than what the devices can do.
+        backends=frozenset({Backend.SNMP, Backend.NSDP}) | _CLI_BACKENDS,
     ),
     Operation(
         "upload_certificate",

{python_netgear_switch_library-0.0.post346 → python_netgear_switch_library-0.0.post348}/src/netgear_switch/nsdp_write.py

@@ -33,6 +33,7 @@ from .models import VlanMode
 from .nsdp_read import AsyncNsdpReader, NsdpReader
 from .protocols.nsdp.protocol import Tag
 from .protocols.nsdp.write import (
+    hostname_tlv,
     ipv4_tlv,
     pvid_tlv,
     vlan_destroy_tlv,
@@ -132,6 +133,28 @@ class NsdpWriter:
     def _vlan(self, vlan: int) -> VLANInfo | None:
         return next((v for v in self._reader.get_vlans() if v.vlan_id == vlan), None)
 
+    def set_hostname(self, name: str, *, force: bool = False) -> None:
+        """Set the switch's host name over NSDP (tag 0x0003).
+
+        The Plus family's only write route for this: those switches have no SNMP
+        agent and no CLI, so without this they cannot be renamed at all.
+
+        Not force-gated -- renaming cannot strand a switch, and it is reversible
+        by writing the old name back. Verified by re-reading the tag.
+        """
+        del force  # accepted for a uniform writer signature; nothing to gate
+        if not name.strip():
+            raise ValueError("hostname must not be empty")
+        before = self._reader.get_hostname()
+        self.client.write([hostname_tlv(name)], password=self._password)
+        after = self._reader.get_hostname()
+        if after != name:
+            raise WriteVerificationError(
+                f"host name is {after!r} after writing {name!r}",
+                before=before,
+                after=after,
+            )
+
     def set_pvid(self, port: int, vlan: int, *, force: bool = False) -> None:
         # LIVE-VERIFIED over NSDP v2 on a GS110EMX (fw 1.0.2.8): PORT_PVID
         # (0x3000) IS writable (the reference spec's READ-ONLY marking was

{python_netgear_switch_library-0.0.post346 → python_netgear_switch_library-0.0.post348}/src/netgear_switch/protocols/nsdp/write.py

@@ -157,6 +157,17 @@ def port_name_tlv(port: int, name: str) -> TLVEntry:
     return TLVEntry(Tag.PORT_NAME, bytes([port]) + name.encode("utf-8"))
 
 
+def hostname_tlv(name: str) -> TLVEntry:
+    """Host-name write TLV (tag 0x0003), the same shape the read decodes.
+
+    The read side is measured -- ``parsers`` decodes this tag as plain text and
+    three live GS110EMX report their names through it -- so the write is that
+    encoding with nothing added: the bare name, ASCII, no length prefix and no
+    port byte (unlike ``port_name_tlv`` above, whose tag is indexed by port).
+    """
+    return TLVEntry(Tag.HOSTNAME, name.encode("ascii"))
+
+
 def ipv4_tlv(tag: Tag, dotted: str) -> TLVEntry:
     return TLVEntry(tag, socket.inet_aton(dotted))
 

{python_netgear_switch_library-0.0.post346 → python_netgear_switch_library-0.0.post348}/src/netgear_switch/snmp_write.py

@@ -707,6 +707,46 @@ class SnmpWriter:
                 after=after,
             )
 
+    def set_syslog_enabled(self, enabled: bool, *, force: bool = False) -> None:
+        """Turn remote syslog on or off.
+
+        Writes the vendor logging admin-mode column (``<base>.14.1.4.1.0``),
+        whose enum is ``1 = enabled, 2 = disabled`` -- established from captured
+        CLI rather than assumed, see ``oids.VendorOids.syslog_admin_mode``.
+
+        WRITABILITY MEASURED 2026-08-02 by SETting each switch the value it
+        already held, which cannot change device state but still distinguishes a
+        writable column from a read-only one: m4300-24x (10.1.5.13), gsm7252ps
+        (10.1.5.22) and gsm7228ps (10.1.5.11) all accepted it.
+
+        Deliberately narrower than ``get_syslog`` reads. Adding or removing a
+        COLLECTOR means creating a row in the host table, which needs a
+        row-status write that has not been driven against hardware; offering it
+        here on the strength of the read alone would be the inference this
+        project refuses.
+
+        Not force-gated: toggling log delivery cannot strand a switch and is
+        reversible by writing the old value back.
+        """
+        del force  # accepted for a uniform writer signature; nothing to gate
+        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"
+            )
+        vo = oids.vendor_oids(self.model)
+        before = self._reader.get_syslog()
+        self.client.set_many(
+            [SetVarbind(vo.syslog_admin_mode, 1 if enabled else 2, "i")]
+        )
+        after = self._reader.get_syslog()
+        if after.enabled != enabled:
+            raise WriteVerificationError(
+                f"syslog enabled is {after.enabled} after writing {enabled}",
+                before=before,
+                after=after,
+            )
+
     def set_hostname(self, name: str, *, force: bool = False) -> None:
         """Set the switch's host name via the standard MIB-II ``sysName``.
 
@@ -1112,6 +1152,27 @@ class AsyncSnmpWriter:
                 after=after,
             )
 
+    async def set_syslog_enabled(self, enabled: bool, *, force: bool = False) -> None:
+        """Async twin of ``SnmpWriter.set_syslog_enabled`` -- see there."""
+        del force  # accepted for a uniform writer signature; nothing to gate
+        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"
+            )
+        vo = oids.vendor_oids(self.model)
+        before = await self._reader.get_syslog()
+        await self.client.set_many(
+            [SetVarbind(vo.syslog_admin_mode, 1 if enabled else 2, "i")]
+        )
+        after = await self._reader.get_syslog()
+        if after.enabled != enabled:
+            raise WriteVerificationError(
+                f"syslog enabled is {after.enabled} after writing {enabled}",
+                before=before,
+                after=after,
+            )
+
     async def set_hostname(self, name: str, *, force: bool = False) -> None:
         """Async twin of ``SnmpWriter.set_hostname`` -- see there."""
         del force  # accepted for a uniform writer signature; nothing to gate

{python_netgear_switch_library-0.0.post346 → python_netgear_switch_library-0.0.post348}/src/netgear_switch/sync_api.py

@@ -503,9 +503,7 @@ class SyncSwitch:
         has -- the actionable information the old silent fallback destroyed.
         """
         if requested is None:
-            others = sorted(
-                b.name for b in self.model.backends if b is not chosen
-            )
+            others = sorted(b.name for b in self.model.backends if b is not chosen)
             hint = (
                 f"; pass backend=Backend.<{'|'.join(others)}> to use another backend"
                 if others
@@ -814,6 +812,21 @@ class SyncSwitch:
             lambda w: w.clear_poe_fault(port, force=force, timeouts=timeouts), backend
         )
 
+    def set_syslog_enabled(
+        self,
+        enabled: bool,
+        *,
+        force: bool = False,
+        backend: Backend | None = None,
+    ) -> None:
+        """Turn remote syslog on or off.
+
+        Served over SNMP. Adding or removing a COLLECTOR is a separate,
+        unbuilt operation -- it needs a row-status write that has not
+        been driven against hardware.
+        """
+        self._write(lambda w: w.set_syslog_enabled(enabled, force=force), backend)
+
     def set_hostname(
         self, name: str, *, force: bool = False, backend: Backend | None = None
     ) -> None:

{python_netgear_switch_library-0.0.post346 → python_netgear_switch_library-0.0.post348}/tests/test_capabilities.py

@@ -97,6 +97,7 @@ _WRITE_ARGS: dict[str, tuple[tuple[Any, ...], dict[str, Any]]] = {
     # `hostname` with no argument is rejected by the device itself and clearing
     # a name is a different command (`no hostname`) that is not implemented.
     "set_hostname": (("capcheck",), {}),
+    "set_syslog_enabled": ((True,), {}),
 }
 
 #: Not driven. ``upload_certificate_scp`` runs a multi-command deploy sequence