{"action":"closed","issue":{"url":"https://api.github.com/repos/sipcapture/homer-ui/issues/53","repository_url":"https://api.github.com/repos/sipcapture/homer-ui","labels_url":"https://api.github.com/repos/sipcapture/homer-ui/issues/53/labels{/name}","comments_url":"https://api.github.com/repos/sipcapture/homer-ui/issues/53/comments","events_url":"https://api.github.com/repos/sipcapture/homer-ui/issues/53/events","html_url":"https://github.com/sipcapture/homer-ui/issues/53","id":130355799,"number":53,"title":"LDAP Groupmembership does not work","user":{"login":"mmonka","id":9898431,"avatar_url":"https://avatars.githubusercontent.com/u/9898431?v=3","gravatar_id":"","url":"https://api.github.com/users/mmonka","html_url":"https://github.com/mmonka","followers_url":"https://api.github.com/users/mmonka/followers","following_url":"https://api.github.com/users/mmonka/following{/other_user}","gists_url":"https://api.github.com/users/mmonka/gists{/gist_id}","starred_url":"https://api.github.com/users/mmonka/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmonka/subscriptions","organizations_url":"https://api.github.com/users/mmonka/orgs","repos_url":"https://api.github.com/users/mmonka/repos","events_url":"https://api.github.com/users/mmonka/events{/privacy}","received_events_url":"https://api.github.com/users/mmonka/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/sipcapture/homer-ui/labels/question","name":"question","color":"cc317c"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2016-02-01T12:57:11Z","updated_at":"2016-02-08T08:56:38Z","closed_at":"2016-02-08T08:56:38Z","body":"To compare uid 'check_filegroup_membership' needs the uid of the user, not the dn.\r\nI check homer 3.5 and there $param['username'] is used; in homer5 $result[0]['dn'].\r\n\r\nAny reason why you compare against dn?\r\n\r\ndiff --git a/api/Authentication/LDAP.php b/api/Authentication/LDAP.php\r\nindex f14760a..7ee4bd9 100644\r\n--- a/api/Authentication/LDAP.php\r\n+++ b/api/Authentication/LDAP.php\r\n@@ -84,7 +84,7 @@ class LDAP extends Authentication {\r\n \r\n \r\n                                      if (defined(\"LDAP_GROUPDN\")) {\r\n-                                        if (!$this->check_filegroup_membership($ds,$result[0]['dn'])) {\r\n+                                        if (!$this->check_filegroup_membership($ds,$result[0]['uid'][0])) {\r\n                                             return false;\r\n                                         }\r\n                                     }"}}