AntoineVe/lan_checker
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 2 Wiki Activity

Fix temperature duplication in MQTT payload

Browse Source 
Temperature now only appears at root level, not in details.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Antoine Van Elstraete
2026-01-26 18:13:49 +01:00
parent 10fad0d9f3
commit 9f3c052e2a
2 changed files with 11 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
checkers/snmp.py
View File

@@ -54,17 +54,15 @@ class SnmpChecker(BaseChecker):
response_time=None response_time=None
) )
else: else:
details = {str(oid): str(val) for oid, val in var_binds} # Only include main OID in details, not temperature
details = {str(var_binds[0][0]): str(var_binds[0][1])}
# Extract temperature if configured # Extract temperature if configured (second OID in response)
if temperature_oid: if temperature_oid and len(var_binds) >= 2:
for oid_key, val in var_binds:
if str(oid_key) == temperature_oid:
try: try:
details["temperature"] = int(val) details["temperature"] = int(var_binds[1][1])
except (ValueError, TypeError): except (ValueError, TypeError):
pass # Ignore if not a valid integer pass # Ignore if not a valid integer
break
return CheckResult( return CheckResult(
success=True, success=True,

5
lan_checker.py
View File

@@ -177,10 +177,11 @@ class LanChecker:
} }
if result.details: if result.details:
payload["details"] = result.details
# Extract temperature for SNMP checks # Extract temperature for SNMP checks
if "temperature" in result.details: if "temperature" in result.details:
payload["temperature"] = result.details["temperature"] payload["temperature"] = result.details.pop("temperature")
if result.details:
payload["details"] = result.details
topic = f"lan_checker/{check['id']}/state" topic = f"lan_checker/{check['id']}/state"
self.mqtt_client.publish(topic, json.dumps(payload), retain=True) self.mqtt_client.publish(topic, json.dumps(payload), retain=True)