Add KeepAlive settings between client and proxy (#41)

* fix: switch server keepAlives

Signed-off-by: fum1h1to <75571344+fum1h1to@users.noreply.github.com>

* fix: test

Signed-off-by: fum1h1to <75571344+fum1h1to@users.noreply.github.com>

* fix: disableKeepAlives comment

Signed-off-by: fum1h1to <75571344+fum1h1to@users.noreply.github.com>

* fix: review

Signed-off-by: fum1h1to <75571344+fum1h1to@users.noreply.github.com>

---------

Signed-off-by: fum1h1to <75571344+fum1h1to@users.noreply.github.com>
Co-authored-by: ly-2045544 <ly-2045544@yahoo-corp.jp>

Author: fsul7o

config/config.go

@@ -64,6 +64,9 @@ type Server struct {
 	// ShutdownDelay represents the delay duration between the health check server shutdown and the api server shutdown.
 	ShutdownDelay string `yaml:"shutdownDelay"`
 
+	// DisableKeepAlives represents whether HTTP keep-alive connections should be disabled between the client and the Provider Sidecar.
+	DisableKeepAlives bool `yaml:"disableKeepAlives"`
+
 	// TLS represents the TLS configuration of the authorization proxy.
 	TLS TLS `yaml:"tls"`
 

config/config_test.go

@@ -77,10 +77,11 @@ func TestNew(t *testing.T) {
 			want: &Config{
 				Version: "v2.0.0",
 				Server: Server{
-					Port:            8082,
-					Timeout:         "10s",
-					ShutdownTimeout: "10s",
-					ShutdownDelay:   "9s",
+					Port:              8082,
+					Timeout:           "10s",
+					ShutdownTimeout:   "10s",
+					ShutdownDelay:     "9s",
+					DisableKeepAlives: true,
 					TLS: TLS{
 						Enable:            true,
 						CertPath:          "test/data/dummyServer.crt",

service/server.go

@@ -126,7 +126,7 @@ func NewServer(opts ...Option) (Server, error) {
 			Addr:    fmt.Sprintf(":%d", s.cfg.Port),
 			Handler: s.srvHandler,
 		}
-		s.srv.SetKeepAlivesEnabled(true)
+		s.srv.SetKeepAlivesEnabled(!s.cfg.DisableKeepAlives)
 		if s.cfg.TLS.Enable {
 			s.srv.TLSConfig = s.tlsConfig
 		}

test/data/example_config.yaml

@@ -5,6 +5,7 @@ server:
   timeout: 10s
   shutdownTimeout: 10s
   shutdownDelay: 9s
+  disableKeepAlives: true
   tls:
     enable: true
     certPath: "test/data/dummyServer.crt"