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 1 commit 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.

Tested on stable release, with saslauthd v2.1.28. LDAP tested against FreeIPA v4.11.1. Final verification of intended outcome (Successful group filtering) pending. 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)

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.

None yet

1 participant