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

Merge pull request #661 from aledbf/update-client-go

Browse source 
Avoid running nginx if the configuration file is empty
This commit is contained in:
Manuel Alejandro de Brito Fontes 2017-04-26 23:58:15 -03:00 committed by GitHub
commit e911f20405
1 changed files with 3 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -334,6 +334,9 @@ func (n NGINXController) DefaultIngressClass() string {
// testTemplate checks if the NGINX configuration inside the byte array is valid
// running the command "nginx -t" using a temporal file.
func (n NGINXController) testTemplate(cfg []byte) error {
if len(cfg) == 0 {
return fmt.Errorf("invalid nginx configuration (empty)")
}
tmpfile, err := ioutil.TempFile("", "nginx-cfg")
if err != nil {
return err