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

Merge pull request #272 from aledbf/refactor-annotation-parsers

Browse source 
Fix error getting class information from Ingress annotations
This commit is contained in:
Manuel Alejandro de Brito Fontes 2017-02-16 07:35:34 -03:00 • committed by GitHub
commit 111f338fa3
7 changed files with 46 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

6
core/pkg/ingress/annotations/authreq/main.go
View file

@ -92,7 +92,11 @@ func (a authReq) Parse(ing *extensions.Ingress) (interface{}, error) {
return nil, ing_errors.NewLocationDenied("invalid url host")
}
m, _ := parser.GetStringAnnotation(authMethod, ing)
m, err := parser.GetStringAnnotation(authMethod, ing)
if err != nil {
return nil, err
}
if len(m) != 0 && !validMethod(m) {
return nil, ing_errors.NewLocationDenied("invalid HTTP method")
}