Merge pull request #768 from prashanthpai/server-timeout

http: Introduce server timeouts
gluster · May 23, 2018 · 199dcb8 · 199dcb8
2 parents 09f1358 + b5e2b49
commit 199dcb8
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/glusterd2/servers/rest/rest.go b/glusterd2/servers/rest/rest.go
@@ -21,6 +21,12 @@ import (
 	config "github.com/spf13/viper"
 )
 
+const (
+	httpReadTimeout  = 10
+	httpWriteTimeout = 30
+	maxHeaderBytes   = 1 << 13 // 8KB
+)
+
 // GDRest is the GlusterD Rest server
 type GDRest struct {
 	Routes   *mux.Router
@@ -50,7 +56,11 @@ func NewMuxed(m cmux.CMux) *GDRest {
 
 	rest := &GDRest{
 		Routes: mux.NewRouter(),
-		server: &http.Server{},
+		server: &http.Server{
+			ReadTimeout:    httpReadTimeout * time.Second,
+			WriteTimeout:   httpWriteTimeout * time.Second,
+			MaxHeaderBytes: maxHeaderBytes,
+		},
 		stopCh: make(chan struct{}),
 	}