Add check if there are any warrnings in the NGINX config validation (#…

…257)
nginx · Mar 23, 2023 · d5bc91f · d5bc91f
1 parent 2c252a2
commit d5bc91f
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/src/core/nginx.go b/src/core/nginx.go
@@ -269,6 +269,10 @@ func (n *NginxBinaryType) ValidateConfig(processId, bin, configLocation string,
 		return fmt.Errorf("error running nginx -t -c %v:\n%s", configLocation, response)
 	}
 
+	if strings.Contains(response.String(), "nginx: [warn]") {
+		return fmt.Errorf("error running nginx -t -c %v:\n%s", configLocation, response)
+	}
+
 	log.Infof("Config validated:\n%s", response)
 
 	return nil

diff --git a/test/performance/vendor/github.com/nginx/agent/v2/src/core/nginx.go b/test/performance/vendor/github.com/nginx/agent/v2/src/core/nginx.go