{"action":"created","issue":{"url":"https://api.github.com/repos/puppet-community/puppet-rundeck/issues/84","labels_url":"https://api.github.com/repos/puppet-community/puppet-rundeck/issues/84/labels{/name}","comments_url":"https://api.github.com/repos/puppet-community/puppet-rundeck/issues/84/comments","events_url":"https://api.github.com/repos/puppet-community/puppet-rundeck/issues/84/events","html_url":"https://github.com/puppet-community/puppet-rundeck/issues/84","id":81794868,"number":84,"title":"bindDn define are different in ldap and AD","user":{"login":"SydOps","id":8954908,"avatar_url":"https://avatars.githubusercontent.com/u/8954908?v=3","gravatar_id":"","url":"https://api.github.com/users/SydOps","html_url":"https://github.com/SydOps","followers_url":"https://api.github.com/users/SydOps/followers","following_url":"https://api.github.com/users/SydOps/following{/other_user}","gists_url":"https://api.github.com/users/SydOps/gists{/gist_id}","starred_url":"https://api.github.com/users/SydOps/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SydOps/subscriptions","organizations_url":"https://api.github.com/users/SydOps/orgs","repos_url":"https://api.github.com/users/SydOps/repos","events_url":"https://api.github.com/users/SydOps/events{/privacy}","received_events_url":"https://api.github.com/users/SydOps/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/puppet-community/puppet-rundeck/labels/bug","name":"bug","color":"fc2929"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-05-28T06:56:28Z","updated_at":"2015-06-02T02:21:14Z","closed_at":"2015-05-29T15:11:44Z","body":"Hi\r\n\r\nI found the define for bindDn are different between `templates/_auth_ad.erb` and  `templates/_auth_ldap.erb`\r\n\r\n```\r\ntemplates/_auth_ad.erb\r\n\r\nbindDn=\"<%= @auth_config['active_directory']['bind_dn']%>\"\r\n\r\ntemplates/_auth_ldap.erb\r\nbindDn=\"CN=<%= @auth_config['ldap']['bind_dn']%>,<%= @auth_config['ldap']['user_base_dn'] %>\"\r\n```\r\nBecause of this, I got very strange result in ldpa setting (merged bindDN and userDN), and our ldap tree is not start from `CN=` . Fix is simple, but any reason to do that?\r\n\r\n    bindDn=\"<%= @auth_config['ldap']['bind_dn']%>\"\r\n"},"comment":{"url":"https://api.github.com/repos/puppet-community/puppet-rundeck/issues/comments/107774293","html_url":"https://github.com/puppet-community/puppet-rundeck/issues/84#issuecomment-107774293","issue_url":"https://api.github.com/repos/puppet-community/puppet-rundeck/issues/84","id":107774293,"user":{"login":"rooty0","id":6276644,"avatar_url":"https://avatars.githubusercontent.com/u/6276644?v=3","gravatar_id":"","url":"https://api.github.com/users/rooty0","html_url":"https://github.com/rooty0","followers_url":"https://api.github.com/users/rooty0/followers","following_url":"https://api.github.com/users/rooty0/following{/other_user}","gists_url":"https://api.github.com/users/rooty0/gists{/gist_id}","starred_url":"https://api.github.com/users/rooty0/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rooty0/subscriptions","organizations_url":"https://api.github.com/users/rooty0/orgs","repos_url":"https://api.github.com/users/rooty0/repos","events_url":"https://api.github.com/users/rooty0/events{/privacy}","received_events_url":"https://api.github.com/users/rooty0/received_events","type":"User","site_admin":false},"created_at":"2015-06-02T02:21:14Z","updated_at":"2015-06-02T02:21:14Z","body":"hey @liamjbennett \r\nI don't really understand how this fix https://github.com/puppet-community/puppet-rundeck/commit/57dd07315844a499e832c26ebb3b60d228828ab1 going to resolve this issue. \r\n\r\nOfficial documentation:\r\n```\r\n....\r\nbindDn=\"cn=Manager,dc=example,dc=com\"\r\n...\r\nuserBaseDn=\"ou=People,dc=test1,dc=example,dc=com\"\r\n``` \r\nwhere **bindDn** and **userBaseDn** completely different values\r\nso I don't understand why do we have **userBaseDn** inside **bindDn**. Can we just remove **userBaseDn** from **bindDn** to make it more flexible?\r\nPlease check this fix https://github.com/puppet-community/puppet-rundeck/issues/96"}}