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

Lint nginx code

Browse source
This commit is contained in:
Manuel de Brito Fontes 2017-06-14 23:05:04 -04:00
parent e9871ffaad
commit 92e3b5ba78
4 changed files with 3 additions and 8 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

@ -165,9 +165,6 @@ type NGINXController struct {
cmdArgs []string
watchClass string
namespace string
stats *statsCollector
statusModule statusModule

4
controllers/nginx/pkg/config/config.go
View file

@ -177,7 +177,7 @@ type Configuration struct {
// Enable json escaping
// http://nginx.org/en/docs/http/ngx_http_log_module.html#log_format
LogFormatEscapeJson bool `json:"log-format-escape-json,omitempty"`
LogFormatEscapeJSON bool `json:"log-format-escape-json,omitempty"`
// Customize upstream log_format
// http://nginx.org/en/docs/http/ngx_http_log_module.html#log_format
@ -315,7 +315,7 @@ func NewDefault() Configuration {
KeepAlive: 75,
KeepAliveRequests: 100,
LargeClientHeaderBuffers: "4 8k",
LogFormatEscapeJson: false,
LogFormatEscapeJSON: false,
LogFormatStream: logFormatStream,
LogFormatUpstream: logFormatUpstream,
MaxWorkerConnections: 16384,

2
controllers/nginx/pkg/template/template.go
View file

@ -24,7 +24,6 @@ import (
"net"
"os"
"os/exec"
"regexp"
"strings"
text_template "text/template"
@ -392,7 +391,6 @@ func isLocationAllowed(input interface{}) bool {
}
var (
nonAlpha = regexp.MustCompile("[^a-zA-Z0-9]+")
denyPathSlugMap = map[string]string{}
)

2
controllers/nginx/rootfs/etc/nginx/template/nginx.tmpl
View file

@ -93,7 +93,7 @@ http {
# disable warnings
uninitialized_variable_warn off;
log_format upstreaminfo {{ if $cfg.LogFormatEscapeJson }}escape=json {{ end }}'{{ buildLogFormatUpstream $cfg }}';
log_format upstreaminfo {{ if $cfg.LogFormatEscapeJSON }}escape=json {{ end }}'{{ buildLogFormatUpstream $cfg }}';
{{/* map urls that should not appear in access.log */}}
{{/* http://nginx.org/en/docs/http/ngx_http_log_module.html#access_log */}}