From 7f4aaed1d59b7b8d4e61bc7f2640b126ba6a9166 Mon Sep 17 00:00:00 2001 From: Simon Law Date: Mon, 12 May 2025 12:53:55 -0700 Subject: [PATCH] cmd/derpprobe: exit with non-zero status if --once fails (#15926) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `cmd/derpprobe --once` didn’t respect the convention of non-zero exit status for a failed run. It would always exit zero (i.e. success), even. This patch fixes that, but only for `--once` mode. Fixes: #15925 Signed-off-by: Simon Law --- cmd/derpprobe/derpprobe.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/cmd/derpprobe/derpprobe.go b/cmd/derpprobe/derpprobe.go index 899838462..2723a31ae 100644 --- a/cmd/derpprobe/derpprobe.go +++ b/cmd/derpprobe/derpprobe.go @@ -9,6 +9,7 @@ import ( "fmt" "log" "net/http" + "os" "sort" "time" @@ -75,6 +76,9 @@ func main() { for _, s := range st.bad { log.Printf("bad: %s", s) } + if len(st.bad) > 0 { + os.Exit(1) + } return }