mdebray/tvl-depot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(tvix/nar-bridge): properly check error during HTTP shutdown

Browse source 
We didn't bind err := here.

Change-Id: I779d7ba5660ba4b126a4a0db7fe044bf94425855
Reviewed-on: https://cl.tvl.fyi/c/depot/+/9547
Reviewed-by: Connor Brewster <cbrewster@hey.com>
Autosubmit: flokli <flokli@flokli.de>
Tested-by: BuildkiteCI
This commit is contained in:
Florian Klink 2023-10-05 17:41:03 +03:00 committed by flokli
parent 96c0b3f069
commit 7e9da4ddef
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
tvix/nar-bridge/cmd/nar-bridge-http/main.go
View file

@ -68,7 +68,7 @@ func main() {
timeoutCtx, cancel := context.WithTimeout(context.Background(), 30*time.Second)
defer cancel()
if s.Shutdown(timeoutCtx); err != nil {
if err := s.Shutdown(timeoutCtx); err != nil {
log.WithError(err).Warn("failed to shutdown")
os.Exit(1)
}