Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

prosody/rootfs/defaults/saslauthd.conf: Append extended LDAP variables #1798

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

le-firehawk
Copy link

@le-firehawk le-firehawk commented May 3, 2024

Include variables from saslauthd documentation here to achieve basic LDAP group membership filtering.

Changes + necessary syntax for FreeIPA compatibility included in env.example.

Tested on stable release, with saslauthd v2.1.28. LDAP tested against FreeIPA v4.11.1. See #1797 for more details.

Peripheral changes:

  • LDAP_START_TLS check was outside PROSODY_AUTH_TYPE eq "LDAP" check. Relocate within
  • Adjust indentation of {{ if }} and {{ end }} statements to reflect their nested depth
  • Cluster variables by function (LDAP bind authentication, LDAP filtering, etc)

env.example Outdated Show resolved Hide resolved
env.example Outdated Show resolved Hide resolved
env.example Show resolved Hide resolved
@le-firehawk le-firehawk force-pushed the master branch 4 times, most recently from 516e291 to 85bff66 Compare June 6, 2024 10:36
prosody/rootfs/defaults/saslauthd.conf Outdated Show resolved Hide resolved
prosody/rootfs/defaults/saslauthd.conf Outdated Show resolved Hide resolved
prosody/rootfs/defaults/saslauthd.conf Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants