Merge pull request #18 from qustavo/fix_leaks

fix leaks
basecamp · Sep 23, 2024 · 726d173 · 726d173
2 parents 556aa40 + 1a660f7
commit 726d173
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/internal/server/health_check.go b/internal/server/health_check.go
@@ -75,6 +75,7 @@ func (hc *HealthCheck) check() {
  hc.consumer.HealthCheckCompleted(false)
  return
  }
+ defer resp.Body.Close()
 
  if resp.StatusCode < 200 || resp.StatusCode > 299 {
  slog.Info("Healthcheck failed", "status", resp.StatusCode)

diff --git a/internal/server/router.go b/internal/server/router.go
@@ -54,6 +54,7 @@ func (r *Router) RestoreLastSavedState() error {
  slog.Error("Failed to restore saved state", "path", r.statePath, "error", err)
  return err
  }
+ defer f.Close()
 
  var services []*Service
  err = json.NewDecoder(f).Decode(&services)