opendefensecloud / solution-arsenal / 29092248515

81%

Build Type

push

github

Committed by web-flow

Commit Message

fix(discovery): use errors.Is for http.ErrServerClosed check (#693)

## What
Use `errors.Is` instead of a direct `!=` comparison when checking for
`http.ErrServerClosed` in the webhook server's serve loop. It ain't much
but it's honest work 😄

## Why
Direct equality comparisons on sentinel errors break if the error is
ever wrapped. `errors.Is` is the standard, wrap-safe way to check
against a sentinel error

## Testing
`go build ./pkg/discovery/...` and `go vet
./pkg/discovery/webhook/...`and also executed `make test`

## Checklist
- [x] Tests added/updated
- [x] No breaking changes (or upgrade path documented above)
- [x] Readable commit history (squashed and cleaned up as desired)
- [ ] AI code review considered and comments resolved

Coverage Stats

1 of 1 new or added line in 1 file covered. (100.0%)

13 existing lines in 5 files now uncovered.

4051 of 5343 relevant lines covered (75.82%)

39.26 hits per line