Skip to content

Commit

Permalink
Fix proxy request buffering default configuration
Browse files Browse the repository at this point in the history
  • Loading branch information
aledbf committed Sep 14, 2017
1 parent 6cca4c5 commit 74c09ca
Showing 1 changed file with 9 additions and 8 deletions.
17 changes: 9 additions & 8 deletions core/pkg/ingress/controller/controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -1078,14 +1078,15 @@ func (ic *GenericController) createServers(data []interface{},

bdef := ic.GetDefaultBackend()
ngxProxy := proxy.Configuration{
BodySize: bdef.ProxyBodySize,
ConnectTimeout: bdef.ProxyConnectTimeout,
SendTimeout: bdef.ProxySendTimeout,
ReadTimeout: bdef.ProxyReadTimeout,
BufferSize: bdef.ProxyBufferSize,
CookieDomain: bdef.ProxyCookieDomain,
CookiePath: bdef.ProxyCookiePath,
NextUpstream: bdef.ProxyNextUpstream,
BodySize: bdef.ProxyBodySize,
ConnectTimeout: bdef.ProxyConnectTimeout,
SendTimeout: bdef.ProxySendTimeout,
ReadTimeout: bdef.ProxyReadTimeout,
BufferSize: bdef.ProxyBufferSize,
CookieDomain: bdef.ProxyCookieDomain,
CookiePath: bdef.ProxyCookiePath,
NextUpstream: bdef.ProxyNextUpstream,
RequestBuffering: bdef.ProxyRequestBuffering,
}

defaultPemFileName := fakeCertificatePath
Expand Down

1 comment on commit 74c09ca

@auhlig
Copy link
Contributor

@auhlig auhlig commented on 74c09ca Sep 15, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aledbf Thanks a lot. Noticed that today. I hoped the test would have pointed me to this.

Please sign in to comment.