myron/novacpx
1
0
Fork 0
mirror of https://github.com/myronblair/novacpx synced 2026-06-30 17:50:41 -05:00
Code Issues Packages Projects Releases Wiki Activity

fix: wrap server_stats INSERT in try/catch — SQLite lock was killing stats API

Browse Source 
Concurrent cron writes (collect-stats.php every 5min) caused DB lock errors
that aborted the entire stats response, leaving web/mail/FTP pages empty.
History insert is now non-fatal.
This commit is contained in:
Myron Blair
2026-06-22 04:52:04 +00:00
parent fcde84d2ad
commit 697763f333
1 changed files with 7 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
panel/api/endpoints/system.php
+3 -1
View File
@@ -390,11 +390,13 @@ BASH;
if ($active) $services[$svc] = $active; if ($active) $services[$svc] = $active;
} }
// Persist to DB for history // Persist to DB for history (non-fatal — don't let lock errors kill the stats response)
try {
$db->execute( $db->execute(
"INSERT INTO server_stats (cpu_usage,ram_usage,disk_usage,load_avg) VALUES (?,?,?,?)", "INSERT INTO server_stats (cpu_usage,ram_usage,disk_usage,load_avg) VALUES (?,?,?,?)",
[$cpuPct, $ramPct, $diskPct, $load[0]] [$cpuPct, $ramPct, $diskPct, $load[0]]
); );
} catch (Throwable $_) {}
Response::success([ Response::success([
'cpu' => ['pct' => $cpuPct, 'load' => $load], 'cpu' => ['pct' => $cpuPct, 'load' => $load],