Merge branch 'fix/set-http-server-timeouts' into 'main'

fix: set http server timeouts

See merge request hectorjsmith/fail2ban-prometheus-exporter!91
This commit is contained in:
Hector 2023-06-21 11:09:39 +00:00
commit b32bbeb1cd

View File

@ -3,6 +3,7 @@ package server
import ( import (
"log" "log"
"net/http" "net/http"
"time"
"gitlab.com/hectorjsmith/fail2ban-prometheus-exporter/cfg" "gitlab.com/hectorjsmith/fail2ban-prometheus-exporter/cfg"
"gitlab.com/hectorjsmith/fail2ban-prometheus-exporter/collector/textfile" "gitlab.com/hectorjsmith/fail2ban-prometheus-exporter/collector/textfile"
@ -26,7 +27,14 @@ func StartServer(
svrErr := make(chan error) svrErr := make(chan error)
go func() { go func() {
svrErr <- http.ListenAndServe(appSettings.MetricsAddress, nil) httpServer := &http.Server{
Addr: appSettings.MetricsAddress,
ReadHeaderTimeout: 10 * time.Second,
ReadTimeout: 10 * time.Second,
WriteTimeout: 10 * time.Second,
IdleTimeout: 30 * time.Second,
}
svrErr <- httpServer.ListenAndServe()
}() }()
log.Print("ready") log.Print("ready")
return svrErr return svrErr