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

Expand config options to allow custom LDAP filter, samba mode #5

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

Conversation

chaos95
Copy link

@chaos95 chaos95 commented Oct 12, 2022

Add new config options (documented in README.md):

  • LSSP_ACCOUNT_FILTER: Sets the $ldap_filter config value for LSSP, defaults to the previously-hard-coded filter value of (&(objectClass=person)($ldap_login_attribute={login}))
  • LDAP_SAMBA_MODE: Allows enabling LSSP's samba mode, which causes it to set the sambaNTPassword/sambaLMPassword attributes when changing the userPassword. Defaults to the previously-hard-coded value of false.
  • SMTP_REQUIRESAUTH: This isn't directly user-set, but the intention is to allow SMTP to be configured without authentication - this internal config value will set LSSP's $mail_smtp_auth to false if no SMTP_USER and SMTP_PASS are set, allowing unauthenticated SMTP.
    • This matches the behaviour described for the SMTP config in README.md, which previously was leaving $mail_smtp_auth enabled even when the credentials were unset.

Fix a minor syntax error in assets/scripts/functions that was putting errors in the log output of the container on startup.

fmod-morgan and others added 7 commits March 18, 2021 11:29
- Default account filter assumes the objectClass of user entries, which
  doesn't suit AD or RFC2807bis directories. This can now be set using
  the environment variable LSSP_ACCOUNT_FILTER, as with other config
  settings.
- For users with open relays (e.g. on internal docker networks),
  configure PHPMailer to disable authentication if no SMTP_USER and
  SMTP_PASS are supplied.
- Add LDAP_SAMBA_MODE default to env-defaults
- Add samba_mode setting in config.inc.php
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.

2 participants