copyblocks: Satisfy and enable gosec rule G114 (#10330)

Signed-off-by: Arve Knudsen <[email protected]>

.golangci.yml

@@ -65,6 +65,7 @@ linters-settings:
   gosec:
     includes:
       - G108
+      - G114
 
 run:
   timeout: 10m

pkg/mimirtool/commands/alerts.go

@@ -351,7 +351,12 @@ func (a *AlertCommand) verifyConfig(_ *kingpin.ParseContext) error {
 	))
 
 	go func() {
-		log.Fatal(http.ListenAndServe(":9090", nil))
+		server := http.Server{
+			Addr:         ":9090",
+			ReadTimeout:  10 * time.Second,
+			WriteTimeout: 10 * time.Second,
+		}
+		log.Fatal(server.ListenAndServe())
 	}()
 
 	ctx := context.Background()

pkg/mimirtool/commands/loadgen.go

@@ -121,8 +121,12 @@ func (c *LoadgenCommand) run(_ *kingpin.ParseContext) error {
 
 	http.Handle("/metrics", promhttp.Handler())
 	go func() {
-		err := http.ListenAndServe(c.metricsListenAddress, nil)
-		if err != nil {
+		server := http.Server{
+			Addr:         c.metricsListenAddress,
+			ReadTimeout:  10 * time.Second,
+			WriteTimeout: 10 * time.Second,
+		}
+		if err := server.ListenAndServe(); err != nil {
 			logrus.WithError(err).Errorln("metrics listener failed")
 		}
 	}()

tools/copyblocks/main.go

@@ -167,8 +167,12 @@ func main() {
 	go func() {
 		level.Info(logger).Log("msg", "HTTP server listening on "+cfg.httpListen)
 		http.Handle("/metrics", promhttp.Handler())
-		err := http.ListenAndServe(cfg.httpListen, nil)
-		if err != nil {
+		server := &http.Server{
+			Addr:         cfg.httpListen,
+			ReadTimeout:  10 * time.Second,
+			WriteTimeout: 10 * time.Second,
+		}
+		if err := server.ListenAndServe(); err != nil {
 			level.Error(logger).Log("msg", "failed to start HTTP server")
 			os.Exit(1)
 		}

tools/trafficdump/main.go

@@ -51,7 +51,12 @@ func main() {
 	if *httpServer != "" {
 		go func() {
 			log.Println("HTTP server running on", *httpServer)
-			log.Println(http.ListenAndServe(*httpServer, nil))
+			server := &http.Server{
+				Addr:         *httpServer,
+				ReadTimeout:  10 * time.Second,
+				WriteTimeout: 10 * time.Second,
+			}
+			log.Println(server.ListenAndServe())
 		}()
 	}