kosmos/chef
kosmos/chef
8
3
Fork 0
Code Issues 29 Pull Requests 2 Projects 2 Activity

Enable LDAP for the XMPP 5apps.com vhost #132

Manually merged
raucao merged 7 commits from feature/123-ejabberd_5apps into master 2020-02-17 14:41:05 +00:00
Conversation 4 Commits 7 Files Changed 4 +1
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 55eb95ae73 - Show all commits

1
site-cookbooks/kosmos-ejabberd/templates/ejabberd.yml.erb
View File

@ -25,6 +25,7 @@ host_config:
ldap_rootdn: "cn=xmpp,ou=<%= host[:name] %>,<%= @ldap_base %>"
ldap_password: "<%= host[:ldap_password] %>"
ldap_encrypt: <%= @ldap_encryption_type %>
ldap_tls_verify: hard # when TLS is enabled, don't proceed if a cert is invalid
ldap_base: "ou=<%= host[:name] %>,<%= @ldap_base %>"
ldap_filter: "(nsRole=cn=xmpp_role,ou=<%= host[:name] %>,<%= @ldap_base %>)"
<% end -%>
raucao commented 2020-02-13 23:40:07 +00:00
Outdated
Review
Copy Link

Why should we add content to the main config file instead of adding separate files for every vhost?

Why should we add content to the main config file instead of adding separate files for every vhost?
greg commented 2020-02-14 09:11:33 +00:00
Outdated
Review
Copy Link

I was considering using different files for the vhosts, but then I could not find a way to define the hosts all at once (https://docs.ejabberd.im/admin/configuration/#host-names), so it didn't seem to make much sense to split the config. I'm going to look into what's possible

I was considering using different files for the vhosts, but then I could not find a way to define the hosts all at once (https://docs.ejabberd.im/admin/configuration/#host-names), so it didn't seem to make much sense to split the config. I'm going to look into what's possible
raucao commented 2020-02-14 16:07:11 +00:00
Outdated
Review
Copy Link

So just because of a single additional line per host it doesn't make sense to split out all the rest?

I don't quite follow to be honest.

So just because of a single additional line per host it doesn't make sense to split out all the rest? I don't quite follow to be honest.