DevFW-CICD/ingress-nginx-helm
14
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #696 from donaldguy/nginx-no-sni

Browse source 
[nginx] pass non-SNI TLS hello to default backend, Fixes #693
This commit is contained in:
Manuel Alejandro de Brito Fontes 2017-05-09 14:07:02 -03:00 committed by GitHub
commit 433220de3c
2 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
controllers/nginx/pkg/cmd/controller/nginx.go
View file

@ -83,7 +83,13 @@ func newNGINXController() ingress.Controller {
configmap: &api_v1.ConfigMap{}, configmap: &api_v1.ConfigMap{},
isIPV6Enabled: isIPv6Enabled(), isIPV6Enabled: isIPv6Enabled(),
resolver: h, resolver: h,
proxy: &proxy{}, proxy: &proxy{
Default: &server{
Hostname: "localhost",
IP: "127.0.0.1",
Port: 442,
},
},
} }
listener, err := net.Listen("tcp", ":443") listener, err := net.Listen("tcp", ":443")

6
controllers/nginx/pkg/cmd/controller/tcp.go
View file

@ -27,11 +27,7 @@ func (p *proxy) Get(host string) *server {
} }
} }
return &server{ return p.Default
Hostname: "localhost",
IP: "127.0.0.1",
Port: 442,
}
} }
func (p *proxy) Handle(conn net.Conn) { func (p *proxy) Handle(conn net.Conn) {