unbound-dhcp-leases-bridge: Don't parse any inactive leases

Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
This commit is contained in:
Michael Tremer
2024-05-10 17:47:44 +01:00
parent b79c0fc4ff
commit dac4464e91

View File

@@ -296,9 +296,9 @@ class UnboundDHCPLeasesBridge(object):
log.debug("Skipping lease for which a static host exists: %s" % lease) log.debug("Skipping lease for which a static host exists: %s" % lease)
return return
# Don't add inactive or expired leases # Don't add expired leases
elif not lease.active or lease.expired: elif lease.expired:
log.debug("Skipping inactive or expired lease: %s" % lease) log.debug("Skipping expired lease: %s" % lease)
return return
# Remove any previous leases # Remove any previous leases
@@ -437,6 +437,9 @@ class DHCPLeases(object):
if not "hardware" in properties: if not "hardware" in properties:
continue continue
# Skip inactive leases
elif not properties.get("binding", "state active"):
continue
lease = Lease(ipaddr, properties) lease = Lease(ipaddr, properties)
leases.append(lease) leases.append(lease)
@@ -556,9 +559,6 @@ class Lease(object):
}) })
@property @property
def active(self):
return self.binding_state == "active"
@property
def hostname(self): def hostname(self):
hostname = self._properties.get("client-hostname") hostname = self._properties.get("client-hostname")