fix: SQLite WAL mode + log all swallowed 500 errors

- internal/db/db.go: Use _pragma syntax so modernc.org/sqlite actually
  applies busy_timeout(5000) and journal_mode(WAL). Eliminates SQLITE_BUSY
  500s when concurrent reads hit a writer holding the DELETE-mode lock.
- internal/api/handlers_*.go: Add slog.Error before every writeError with
  StatusInternalServerError so real errors appear in logs (30+ sites across
  handlers_jobs, handlers_machines, handlers_syncpairs, handlers_sshkeys).
This commit is contained in:
2026-07-08 23:55:55 -04:00
parent d0fd0e994f
commit 1734167f83
5 changed files with 31 additions and 1 deletions
+7
View File
@@ -57,6 +57,7 @@ func (h *JobHandler) List(w http.ResponseWriter, r *http.Request) {
repo := models.NewJobLogRepository(h.db) repo := models.NewJobLogRepository(h.db)
jobs, total, err := repo.GetAllFiltered(limit, offset, syncPairID, status, triggerType, from, to) jobs, total, err := repo.GetAllFiltered(limit, offset, syncPairID, status, triggerType, from, to)
if err != nil { if err != nil {
slog.Error("failed to fetch jobs", "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch jobs") writeError(w, http.StatusInternalServerError, "failed to fetch jobs")
return return
} }
@@ -83,6 +84,7 @@ func (h *JobHandler) Get(w http.ResponseWriter, r *http.Request) {
return return
} }
if err != nil { if err != nil {
slog.Error("failed to fetch job", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch job") writeError(w, http.StatusInternalServerError, "failed to fetch job")
return return
} }
@@ -103,6 +105,7 @@ func (h *JobHandler) Cancel(w http.ResponseWriter, r *http.Request) {
return return
} }
if err != nil { if err != nil {
slog.Error("failed to fetch job", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch job") writeError(w, http.StatusInternalServerError, "failed to fetch job")
return return
} }
@@ -146,6 +149,7 @@ func (h *JobHandler) TriggerRun(w http.ResponseWriter, r *http.Request) {
jobID, err := h.engine.CreateJob(pairID, "manual") jobID, err := h.engine.CreateJob(pairID, "manual")
if err != nil { if err != nil {
slog.Error("failed to create job", "pair_id", pairID, "error", err)
writeError(w, http.StatusInternalServerError, "failed to create job") writeError(w, http.StatusInternalServerError, "failed to create job")
return return
} }
@@ -162,6 +166,7 @@ func (h *JobHandler) TriggerRun(w http.ResponseWriter, r *http.Request) {
jobRepo := models.NewJobRepository(h.db) jobRepo := models.NewJobRepository(h.db)
j, err := jobRepo.GetByID(jobID) j, err := jobRepo.GetByID(jobID)
if err != nil { if err != nil {
slog.Error("failed to fetch created job", "job_id", jobID, "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch created job") writeError(w, http.StatusInternalServerError, "failed to fetch created job")
return return
} }
@@ -184,6 +189,7 @@ func (h *JobHandler) GetLog(w http.ResponseWriter, r *http.Request) {
logRepo := models.NewJobLogRepository(h.db) logRepo := models.NewJobLogRepository(h.db)
logs, err := logRepo.GetByJobID(id, limit, offset) logs, err := logRepo.GetByJobID(id, limit, offset)
if err != nil { if err != nil {
slog.Error("failed to fetch logs", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch logs") writeError(w, http.StatusInternalServerError, "failed to fetch logs")
return return
} }
@@ -207,6 +213,7 @@ func (h *JobHandler) DownloadLog(w http.ResponseWriter, r *http.Request) {
jobRepo := models.NewJobRepository(h.db) jobRepo := models.NewJobRepository(h.db)
j, err := jobRepo.GetByID(id) j, err := jobRepo.GetByID(id)
if err != nil { if err != nil {
slog.Error("failed to fetch job", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch job") writeError(w, http.StatusInternalServerError, "failed to fetch job")
return return
} }
+7
View File
@@ -30,6 +30,7 @@ func (h *MachineHandler) List(w http.ResponseWriter, r *http.Request) {
repo := models.NewMachineRepository(h.db) repo := models.NewMachineRepository(h.db)
ms, err := repo.GetAll() ms, err := repo.GetAll()
if err != nil { if err != nil {
slog.Error("failed to fetch machines", "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch machines") writeError(w, http.StatusInternalServerError, "failed to fetch machines")
return return
} }
@@ -53,6 +54,7 @@ func (h *MachineHandler) Get(w http.ResponseWriter, r *http.Request) {
return return
} }
if err != nil { if err != nil {
slog.Error("failed to fetch machine", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch machine") writeError(w, http.StatusInternalServerError, "failed to fetch machine")
return return
} }
@@ -106,6 +108,7 @@ func (h *MachineHandler) Create(w http.ResponseWriter, r *http.Request) {
repo := models.NewMachineRepository(h.db) repo := models.NewMachineRepository(h.db)
id, err := repo.Create(m) id, err := repo.Create(m)
if err != nil { if err != nil {
slog.Error("failed to create machine", "name", req.Name, "host", req.Host, "error", err)
writeError(w, http.StatusInternalServerError, "failed to create machine") writeError(w, http.StatusInternalServerError, "failed to create machine")
return return
} }
@@ -150,6 +153,7 @@ func (h *MachineHandler) Update(w http.ResponseWriter, r *http.Request) {
return return
} }
if err != nil { if err != nil {
slog.Error("failed to fetch machine", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch machine") writeError(w, http.StatusInternalServerError, "failed to fetch machine")
return return
} }
@@ -170,6 +174,7 @@ func (h *MachineHandler) Update(w http.ResponseWriter, r *http.Request) {
} }
if err := repo.Update(existing); err != nil { if err := repo.Update(existing); err != nil {
slog.Error("failed to update machine", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to update machine") writeError(w, http.StatusInternalServerError, "failed to update machine")
return return
} }
@@ -189,6 +194,7 @@ func (h *MachineHandler) TestWoL(w http.ResponseWriter, r *http.Request) {
return return
} }
if err != nil { if err != nil {
slog.Error("failed to fetch machine", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch machine") writeError(w, http.StatusInternalServerError, "failed to fetch machine")
return return
} }
@@ -238,6 +244,7 @@ func (h *MachineHandler) Delete(w http.ResponseWriter, r *http.Request) {
repo := models.NewMachineRepository(h.db) repo := models.NewMachineRepository(h.db)
if err := repo.Delete(id); err != nil { if err := repo.Delete(id); err != nil {
slog.Error("failed to delete machine", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to delete machine") writeError(w, http.StatusInternalServerError, "failed to delete machine")
return return
} }
+9
View File
@@ -6,6 +6,7 @@ import (
"encoding/json" "encoding/json"
"fmt" "fmt"
"io" "io"
"log/slog"
"net/http" "net/http"
"os" "os"
"path/filepath" "path/filepath"
@@ -32,6 +33,7 @@ func (h *SSHKeyHandler) List(w http.ResponseWriter, r *http.Request) {
machineRepo := models.NewMachineRepository(h.db) machineRepo := models.NewMachineRepository(h.db)
keys, err := repo.GetAll() keys, err := repo.GetAll()
if err != nil { if err != nil {
slog.Error("failed to fetch ssh keys", "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch ssh keys") writeError(w, http.StatusInternalServerError, "failed to fetch ssh keys")
return return
} }
@@ -87,6 +89,7 @@ func (h *SSHKeyHandler) Create(w http.ResponseWriter, r *http.Request) {
keysDir := filepath.Join(h.cfg.SSHDir(), "keys") keysDir := filepath.Join(h.cfg.SSHDir(), "keys")
if err := os.MkdirAll(keysDir, 0700); err != nil { if err := os.MkdirAll(keysDir, 0700); err != nil {
slog.Error("failed to create keys directory", "path", keysDir, "error", err)
writeError(w, http.StatusInternalServerError, "failed to create keys directory") writeError(w, http.StatusInternalServerError, "failed to create keys directory")
return return
} }
@@ -97,6 +100,7 @@ func (h *SSHKeyHandler) Create(w http.ResponseWriter, r *http.Request) {
if req.Generate { if req.Generate {
privPath, _, pubKey, fp, err = sshmanager.GenerateKeyPair(req.Label, keysDir) privPath, _, pubKey, fp, err = sshmanager.GenerateKeyPair(req.Label, keysDir)
if err != nil { if err != nil {
slog.Error("generating key", "label", req.Label, "error", err)
writeError(w, http.StatusInternalServerError, fmt.Sprintf("generating key: %v", err)) writeError(w, http.StatusInternalServerError, fmt.Sprintf("generating key: %v", err))
return return
} }
@@ -113,6 +117,7 @@ func (h *SSHKeyHandler) Create(w http.ResponseWriter, r *http.Request) {
repo := models.NewSSHKeyRepository(h.db) repo := models.NewSSHKeyRepository(h.db)
id, err := repo.Create(req.Label, privPath, pubKey) id, err := repo.Create(req.Label, privPath, pubKey)
if err != nil { if err != nil {
slog.Error("failed to store ssh key", "label", req.Label, "error", err)
writeError(w, http.StatusInternalServerError, "failed to store ssh key") writeError(w, http.StatusInternalServerError, "failed to store ssh key")
return return
} }
@@ -140,6 +145,7 @@ func (h *SSHKeyHandler) Get(w http.ResponseWriter, r *http.Request) {
return return
} }
if err != nil { if err != nil {
slog.Error("failed to fetch ssh key", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch ssh key") writeError(w, http.StatusInternalServerError, "failed to fetch ssh key")
return return
} }
@@ -195,6 +201,7 @@ func (h *SSHKeyHandler) Delete(w http.ResponseWriter, r *http.Request) {
os.Remove(k.PrivateKeyPath + ".pub") os.Remove(k.PrivateKeyPath + ".pub")
} }
if err := repo.Delete(id); err != nil { if err := repo.Delete(id); err != nil {
slog.Error("failed to delete ssh key", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to delete ssh key") writeError(w, http.StatusInternalServerError, "failed to delete ssh key")
return return
} }
@@ -214,6 +221,7 @@ func (h *SSHKeyHandler) DownloadPrivate(w http.ResponseWriter, r *http.Request)
return return
} }
if err != nil { if err != nil {
slog.Error("failed to fetch ssh key", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch ssh key") writeError(w, http.StatusInternalServerError, "failed to fetch ssh key")
return return
} }
@@ -224,6 +232,7 @@ func (h *SSHKeyHandler) DownloadPrivate(w http.ResponseWriter, r *http.Request)
data, err := os.ReadFile(k.PrivateKeyPath) data, err := os.ReadFile(k.PrivateKeyPath)
if err != nil { if err != nil {
slog.Error("failed to read private key", "path", k.PrivateKeyPath, "error", err)
writeError(w, http.StatusInternalServerError, "failed to read private key") writeError(w, http.StatusInternalServerError, "failed to read private key")
return return
} }
+7
View File
@@ -3,6 +3,7 @@ package api
import ( import (
"database/sql" "database/sql"
"encoding/json" "encoding/json"
"log/slog"
"net/http" "net/http"
"regexp" "regexp"
"strconv" "strconv"
@@ -25,6 +26,7 @@ func (h *SyncPairHandler) List(w http.ResponseWriter, r *http.Request) {
repo := models.NewSyncPairRepository(h.db) repo := models.NewSyncPairRepository(h.db)
pairs, err := repo.GetAll() pairs, err := repo.GetAll()
if err != nil { if err != nil {
slog.Error("failed to fetch sync pairs", "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch sync pairs") writeError(w, http.StatusInternalServerError, "failed to fetch sync pairs")
return return
} }
@@ -48,6 +50,7 @@ func (h *SyncPairHandler) Get(w http.ResponseWriter, r *http.Request) {
return return
} }
if err != nil { if err != nil {
slog.Error("failed to fetch sync pair", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch sync pair") writeError(w, http.StatusInternalServerError, "failed to fetch sync pair")
return return
} }
@@ -94,6 +97,7 @@ func (h *SyncPairHandler) Create(w http.ResponseWriter, r *http.Request) {
repo := models.NewSyncPairRepository(h.db) repo := models.NewSyncPairRepository(h.db)
id, err := repo.Create(sp) id, err := repo.Create(sp)
if err != nil { if err != nil {
slog.Error("failed to create sync pair", "name", req.Name, "error", err)
writeError(w, http.StatusInternalServerError, "failed to create sync pair") writeError(w, http.StatusInternalServerError, "failed to create sync pair")
return return
} }
@@ -131,6 +135,7 @@ func (h *SyncPairHandler) Update(w http.ResponseWriter, r *http.Request) {
return return
} }
if err != nil { if err != nil {
slog.Error("failed to fetch sync pair", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to fetch sync pair") writeError(w, http.StatusInternalServerError, "failed to fetch sync pair")
return return
} }
@@ -146,6 +151,7 @@ func (h *SyncPairHandler) Update(w http.ResponseWriter, r *http.Request) {
existing.Enabled = req.Enabled existing.Enabled = req.Enabled
if err := repo.Update(existing); err != nil { if err := repo.Update(existing); err != nil {
slog.Error("failed to update sync pair", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to update sync pair") writeError(w, http.StatusInternalServerError, "failed to update sync pair")
return return
} }
@@ -161,6 +167,7 @@ func (h *SyncPairHandler) Delete(w http.ResponseWriter, r *http.Request) {
repo := models.NewSyncPairRepository(h.db) repo := models.NewSyncPairRepository(h.db)
if err := repo.Delete(id); err != nil { if err := repo.Delete(id); err != nil {
slog.Error("failed to delete sync pair", "id", id, "error", err)
writeError(w, http.StatusInternalServerError, "failed to delete sync pair") writeError(w, http.StatusInternalServerError, "failed to delete sync pair")
return return
} }
+1 -1
View File
@@ -24,7 +24,7 @@ func Open(dbPath string) (*DB, error) {
return nil, err return nil, err
} }
db, err := sql.Open("sqlite", dbPath+"?_journal_mode=WAL&_foreign_keys=ON&_busy_timeout=5000") db, err := sql.Open("sqlite", dbPath+"?_pragma=busy_timeout(5000)&_pragma=journal_mode(WAL)")
if err != nil { if err != nil {
return nil, fmt.Errorf("opening db: %w", err) return nil, fmt.Errorf("opening db: %w", err)
} }