Skip to content

Fix Graphite panic on UDP dial failure - #13

Merged
jdevera merged 1 commit into
mainfrom
fix/graphite-sandbox-panic
Sep 13, 2026
Merged

jdevera merged 1 commit into
mainfrom
fix/graphite-sandbox-panic

Conversation

@jdevera

@jdevera jdevera commented Sep 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • replace the archived graphite-golang client with the existing safe UDP sending path
  • preserve the documented Graphite plaintext format and one-datagram-per-metric behavior
  • return and log transport errors without changing command output or exit status
  • remove the unused Graphite dependency
  • add UDP wire-format and dial-failure regression tests

Sandbox behavior

When the launcher runs where UDP socket creation is denied, Graphite delivery now returns an ordinary operation-not-permitted error. The command keeps its stdout and original exit code. The failure is silent by default and is written to the launcher log when error logging is enabled.

Validation

  • go test ./...
  • go test ./cmd/metrics -count=10
  • go vet ./...
  • go mod tidy -diff
  • reproduced UDP denial in a restricted sandbox and confirmed a successful launcher command still exits 0

Closes #12

@jdevera
jdevera merged commit f7bad75 into main Sep 13, 2026
8 checks passed
@jdevera
jdevera deleted the fix/graphite-sandbox-panic branch September 13, 2026 14:30
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.

Metrics transport failure panics and changes command exit status

1 participant