#3 Fix required groups
This commit is contained in:
parent
ca6aab5629
commit
9b7d526d85
2
build
2
build
|
@ -3,7 +3,7 @@
|
||||||
set -e
|
set -e
|
||||||
|
|
||||||
base='docker.io/tpimenta/nginx-ldap-auth'
|
base='docker.io/tpimenta/nginx-ldap-auth'
|
||||||
version='v1.0.1'
|
version='v1.0.2'
|
||||||
image="$base:$version"
|
image="$base:$version"
|
||||||
|
|
||||||
atexit() {
|
atexit() {
|
||||||
|
|
|
@ -71,7 +71,7 @@ spec:
|
||||||
spec:
|
spec:
|
||||||
serviceAccountName: nginx-ldap-auth
|
serviceAccountName: nginx-ldap-auth
|
||||||
containers:
|
containers:
|
||||||
- image: docker.io/tpimenta/nginx-ldap-auth:v1.0.0
|
- image: docker.io/tpimenta/nginx-ldap-auth:v1.0.2
|
||||||
name: nginx-ldap-auth
|
name: nginx-ldap-auth
|
||||||
command:
|
command:
|
||||||
- "nginx-ldap-auth"
|
- "nginx-ldap-auth"
|
||||||
|
|
2
k8s.yaml
2
k8s.yaml
|
@ -26,7 +26,7 @@ spec:
|
||||||
app: nginx-ldap-auth
|
app: nginx-ldap-auth
|
||||||
spec:
|
spec:
|
||||||
containers:
|
containers:
|
||||||
- image: docker.io/tpimenta/nginx-ldap-auth:v1.0.1
|
- image: docker.io/tpimenta/nginx-ldap-auth:v1.0.2
|
||||||
name: nginx-ldap-auth
|
name: nginx-ldap-auth
|
||||||
command:
|
command:
|
||||||
- "nginx-ldap-auth"
|
- "nginx-ldap-auth"
|
||||||
|
|
|
@ -55,7 +55,7 @@ func (p *Service) validate(username, password string) (bool, error) {
|
||||||
}
|
}
|
||||||
|
|
||||||
if ok || p.required == nil || len(p.required) == 0 {
|
if ok || p.required == nil || len(p.required) == 0 {
|
||||||
return err != nil, nil
|
return err == nil, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
groups, err := p.group.Find(id)
|
groups, err := p.group.Find(id)
|
||||||
|
|
Reference in a new issue