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

Add plugin lint for this change

Browse source
This commit is contained in:
Alex Kursell 2019-03-29 16:36:08 -04:00
parent fdd445cb2a
commit 6cbfd4c7d1
1 changed files with 15 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
cmd/plugin/lints/ingress.go
View file

@ -78,9 +78,24 @@ func GetIngressLints() []IngressLint {
message: "Contains an annotation with the prefix 'nginx.com'. This is a prefix for https://github.com/nginxinc/kubernetes-ingress", message: "Contains an annotation with the prefix 'nginx.com'. This is a prefix for https://github.com/nginxinc/kubernetes-ingress",
f: annotationPrefixIsNginxCom, f: annotationPrefixIsNginxCom,
}, },
{
message: "The x-forwarded-prefix annotation value is a boolean instead of a string",
issue: 3786,
version: "0.24.0",
f: xForwardedPrefixIsBool,
},
} }
} }
func xForwardedPrefixIsBool(ing v1beta1.Ingress) bool {
for name, val := range ing.Annotations {
if strings.HasSuffix(name, "/x-forwarded-prefix") && (val == "true" || val == "false") {
return true
}
}
return false
}
func annotationPrefixIsNginxCom(ing v1beta1.Ingress) bool { func annotationPrefixIsNginxCom(ing v1beta1.Ingress) bool {
for name := range ing.Annotations { for name := range ing.Annotations {
if strings.HasPrefix(name, "nginx.com/") { if strings.HasPrefix(name, "nginx.com/") {