Skip to content

fix: replace silent catch {} with log warnings#375

Merged
kacy merged 4 commits intomainfrom
fix/error-hardening
Apr 2, 2026
Merged

fix: replace silent catch {} with log warnings#375
kacy merged 4 commits intomainfrom
fix/error-hardening

Conversation

@kacy
Copy link
Copy Markdown
Owner

@kacy kacy commented Apr 2, 2026

Summary

  • log warning when api server SO_REUSEADDR or read timeout setsockopt fails
  • log warning when tls proxy fails to write redirect, acme challenge, close_notify, or encrypted data responses
  • log warning when gpu cgroup device file writes fail
  • log warning when deployment status updates fail during batch rollback/pause

11 silent catch {} sites replaced with log.warn(...) across 7 files. These were making production debugging impossible for transient failures.

@kacy kacy merged commit 38ab72b into main Apr 2, 2026
6 of 7 checks passed
@kacy kacy deleted the fix/error-hardening branch April 2, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant