Changing status reason should not trigger a log message

murrant · murrant · commit 25cea11414d2 · 2026-07-06T15:57:32.000-05:00
diff --git a/app/Observers/DeviceObserver.php b/app/Observers/DeviceObserver.php
@@ -41,7 +41,7 @@ public function created(Device $device): void
     public function updated(Device $device): void
     {
         // log up/down status changes
-        if ($device->isDirty(['status', 'status_reason'])) {
+        if ($device->isDirty('status')) {
             $type = $device->status ? 'up' : 'down';
             $reason = $device->status ? $device->getOriginal('status_reason') : $device->status_reason;
             $polled_by = LibrenmsConfig::get('distributed_poller') ? (' by ' . \config('librenms.node_id')) : '';

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ public function created(Device $device): void`
`41`	`41`	`public function updated(Device $device): void`
`42`	`42`	`{`
`43`	`43`	`// log up/down status changes`
`44`		`- if ($device->isDirty(['status', 'status_reason'])) {`
	`44`	`+ if ($device->isDirty('status')) {`
`45`	`45`	`$type = $device->status ? 'up' : 'down';`
`46`	`46`	`$reason = $device->status ? $device->getOriginal('status_reason') : $device->status_reason;`
`47`	`47`	`$polled_by = LibrenmsConfig::get('distributed_poller') ? (' by ' . \config('librenms.node_id')) : '';`