From 76eeec491484a7327b6fb54658970e741b0835fe Mon Sep 17 00:00:00 2001
From: Tomasz Durda <tomekdur@wp.pl>
Date: Tue, 9 Jun 2020 00:39:28 +0200
Subject: [PATCH] Updated LDAP (markdown)

---
 LDAP.md | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/LDAP.md b/LDAP.md
index ea299c0..efc8be5 100644
--- a/LDAP.md
+++ b/LDAP.md
@@ -4,6 +4,8 @@ Custom values can be injected using environment variables, similar to the main c
 
 LDAP configuration is done by [django ldap module](https://django-auth-ldap.readthedocs.io/en/latest/install.html)
 
+Take a note that changing `configuration/ldap_config.py` is strongly discouraged. Netbox container reports barely any python errors
+
 ## Override example
 
 Among others you can use *docker-compose.override.yml* to inject necessary variables. 
@@ -29,7 +31,7 @@ services:
 
 ### Example override file for OpenLDAP
 
-Remember to change `AUTH_LDAP_GROUP_TYPE` to proper value
+Remember to change `AUTH_LDAP_GROUP_TYPE` to proper value.
 
 ```yml
 version: '3.4'
@@ -47,6 +49,7 @@ services:
       AUTH_LDAP_IS_SUPERUSER_DN: "cn=shrek,ou=people,dc=domain,dc=com"
       AUTH_LDAP_USER_SEARCH_ATTR: "cn"
       AUTH_LDAP_GROUP_SEARCH_CLASS: "groupOfUniqueNames"
+      AUTH_LDAP_GROUP_TYPE: "GroupOfUniqueNamesType"
       AUTH_LDAP_ATTR_LASTNAME: "sn"
       AUTH_LDAP_ATTR_FIRSTNAME: "givenName"
       LDAP_IGNORE_CERT_ERRORS: "false"