Skip to content

Commit 6fd8c29

Browse files
committed
update auth-ldap.conf.j2 to optionally check group and add all auth-ldap specific variables to defaults/main.yml
1 parent 4e05902 commit 6fd8c29

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

defaults/main.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -177,8 +177,15 @@ openvpn_use_pam_users: []
177177

178178
# LDAP authentication and configuration (optional)
179179
openvpn_use_ldap: false
180+
openvpn_ldap_server: # ldapserver.example.org or ldap://ldapserver.example.org
180181
openvpn_ldap_tlsenable: 'false'
181182
openvpn_ldap_follow_referrals: 'false'
183+
openvpn_ldap_bind_dn: # cn=administrator,cn=users,dc=ctc,dc=local
184+
openvpn_ldap_bind_password:
185+
openvpn_ldap_base_dn: # dc=ctc,dc=local
186+
openvpn_ldap_search_filter: # sAMAccountName=%u
187+
openvpn_ldap_group_base_dn: # ou=groups,dc=ctc,dc=local if empty fallback to openvpn_ldap_base_dn
188+
openvpn_ldap_group_search_filter: # cn=OpenVPNUsers
182189

183190
# Use simple authentication (default is disabled)
184191
openvpn_simple_auth: false

templates/authentication/auth-ldap.conf.j2

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,10 +45,12 @@
4545
# e.g. "sAMAccountName=%u"
4646
SearchFilter {{ openvpn_ldap_search_filter }}
4747
RequireGroup true
48+
{% if openvpn_ldap_group_search_filter %}
4849
<Group>
49-
BaseDN {{ openvpn_ldap_base_dn }}
50+
BaseDN {{ openvpn_ldap_group_base_dn | default(openvpn_ldap_base_dn) }}
5051
# e.g. "cn=OpenVPNUsers"
5152
SearchFilter {{ openvpn_ldap_group_search_filter }}
5253
MemberAttribute Member
5354
</Group>
55+
{% endif %}
5456
</Authorization>

0 commit comments

Comments
 (0)