Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

Commit

Permalink
Option DisableKeepAlives is a workaround
Browse files Browse the repository at this point in the history
We no longer need this option and we don't use it.
  • Loading branch information
eitu5ami committed Dec 14, 2023
1 parent 5d72949 commit bee4b10
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
5 changes: 2 additions & 3 deletions internal/proxy/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,8 @@ type ProxyConfig struct { // nolint:revive
}

type TargetConnectionHTTP struct {
URL string `yaml:"url"`
Compression bool `yaml:"compression"`
DisableKeepAlives bool `yaml:"disableKeepAlives"`
URL string `yaml:"url"`
Compression bool `yaml:"compression"`
}

type TargetConfigConnection struct {
Expand Down
1 change: 0 additions & 1 deletion internal/proxy/reverseproxy.go
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ func NewReverseProxy(targetConfig TargetConfig, config Config) (*httputil.Revers
Proxy: http.ProxyFromEnvironment,
DialContext: conntrackDialer,
ForceAttemptHTTP2: true,
DisableKeepAlives: targetConfig.Connection.HTTP.DisableKeepAlives,
MaxIdleConns: 100,
IdleConnTimeout: 30 * time.Second,
TLSHandshakeTimeout: 10 * time.Second,
Expand Down

0 comments on commit bee4b10

Please sign in to comment.