silence "Unsolicited response received" again

ConSol-Monitoring · Mar 21, 2024 · 130ee60 · 130ee60
1 parent c3fdf3d
commit 130ee60
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/Changes b/Changes
@@ -2,6 +2,7 @@ This file documents the revision history for check_nsc_web
 
 next:
           - minimum go version is now 1.22
+          - silence "Unsolicited response received" messages
 
 0.7.1    Wed Mar 13 20:30:41 CET 2024
           - remove url parameters from error output

diff --git a/pkg/checknscweb/check.go b/pkg/checknscweb/check.go
@@ -225,8 +225,8 @@ func Check(ctx context.Context, output io.Writer, osArgs []string) int {
 
 		return 3
 	}
-	defer hClient.CloseIdleConnections()
 
+	log.SetOutput(io.Discard) // avoid "Unsolicited response received on idle HTTP channel starting with" messages
 	req, err := buildRequest(ctx, output, queryURL.String(), flags)
 	if err != nil {
 		fmt.Fprintf(output, "UNKNOWN - %s", err.Error())
@@ -251,6 +251,7 @@ func Check(ctx context.Context, output io.Writer, osArgs []string) int {
 
 		return errorExit
 	}
+	hClient.CloseIdleConnections()
 
 	if flags.Verbose {
 		dumpres, err2 := httputil.DumpResponse(res, true)
@@ -261,7 +262,6 @@ func Check(ctx context.Context, output io.Writer, osArgs []string) int {
 		fmt.Fprintf(output, "<<<<<<RESPONSE:\n%s\n<<<<<<\n", dumpres)
 	}
 
-	log.SetOutput(io.Discard)
 	contents, err := extractHTTPResponse(res)
 	if err != nil {
 		fmt.Fprintf(output, "RESPONSE-ERROR: %s\n", err.Error())