A DNS lookup cut short by shutdown is logged as an error #229

New Issue

2026-10-02T07:35:34+02:00

clawbot commented

2026-10-02 07:35:34 +02:00

Found in the trial run for #149.

Stopping the container while a DNS check was running logged:

"level":"ERROR","msg":"failed to lookup records","hostname":"www.bbc.co.uk","error":"context canceled"

Nothing failed: shutdown cut the lookup short, and the README says such a lookup "saves nothing and sends no notification". An error line on an ordinary stop makes a clean stop look like a fault to anyone who watches error-level lines. The port and TLS checks already return quietly when shutdown cuts them short; the DNS lookups do not.

To see it, run docker stop on a container while a DNS check is running. The lines are logged by checkHostname and checkDomain in internal/watcher/watcher.go.

Definition of done

A DNS lookup cut short by shutdown logs nothing at error or warning level.
A lookup that fails for any other reason is still logged at error level.
A test checks both.

Model: opus-5-5

Found in the trial run for https://git.eeqj.de/sneak/dnswatcher/issues/149. Stopping the container while a DNS check was running logged: ``` "level":"ERROR","msg":"failed to lookup records","hostname":"www.bbc.co.uk","error":"context canceled" ``` Nothing failed: shutdown cut the lookup short, and the README says such a lookup "saves nothing and sends no notification". An error line on an ordinary stop makes a clean stop look like a fault to anyone who watches error-level lines. The port and TLS checks already return quietly when shutdown cuts them short; the DNS lookups do not. To see it, run `docker stop` on a container while a DNS check is running. The lines are logged by `checkHostname` and `checkDomain` in `internal/watcher/watcher.go`. ## Definition of done - A DNS lookup cut short by shutdown logs nothing at error or warning level. - A lookup that fails for any other reason is still logged at error level. - A test checks both. Model: opus-5-5

clawbot added this to the 1.0 milestone 2026-10-02 07:35:34 +02:00

clawbot referenced this issue

2026-10-02 07:35:45 +02:00

Trial run of the finished image against real targets before tagging 1.0 #149

clawbot referenced this issue

2026-10-02 07:39:18 +02:00

Review toward 1.0: identify and file all next steps #144

clawbot referenced this issue from a commit

2026-10-02 08:04:53 +02:00

watcher: a lookup cut short by shutdown is not logged as an error (closes #229)

clawbot referenced a pull request that will close this issue

2026-10-02 08:07:44 +02:00

watcher: a lookup cut short by shutdown is not logged as an error (closes #229) #237

clawbot commented

2026-10-02 08:07:50 +02:00

#237: a DNS lookup that shutdown cuts short is no longer logged as an error; a lookup that fails for any other reason, running out of time included, still is.

Model: opus-5-5

https://git.eeqj.de/sneak/dnswatcher/pulls/237: a DNS lookup that shutdown cuts short is no longer logged as an error; a lookup that fails for any other reason, running out of time included, still is. Model: opus-5-5

clawbot referenced this issue from a commit

2026-10-02 08:22:11 +02:00

watcher: a lookup cut short by shutdown is not logged as an error (closes #229)

clawbot referenced this issue from a commit

2026-10-02 08:50:13 +02:00

watcher: a lookup cut short by shutdown is not logged as an error (closes #229)

clawbot closed this issue

2026-10-02 08:56:54 +02:00

clawbot referenced this issue from a commit

2026-10-02 08:56:55 +02:00

watcher: a lookup cut short by shutdown is not logged as an error (closes #229)

clawbot referenced this issue