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

Settings.py or environment Variables #242

Closed
aiveras opened this issue Feb 16, 2020 · 4 comments
Closed

Settings.py or environment Variables #242

aiveras opened this issue Feb 16, 2020 · 4 comments

Comments

@aiveras
Copy link

aiveras commented Feb 16, 2020

Since the new update LDAP no longer works directly. Unfortunately I didn't find anything about this in the documentation:
Shouldn't settings be set with environment variables? Or in settings.py?
It is also not clear if SAML can be set with the environment variables.

@timabbott
Copy link
Member

I don't believe we have a way for SAML to work with environment variables. We recommend using MANUAL_CONFIGURATION with a settings.py file for that.

What issue do you get with LDAP done via environment? In general environment variables are not a super robust way to pass the relatively complex parameters involved in LDAP, so I'm not sure it's possible to make that work in all cases.

But it sounds like you think there's a regression, and if it is, we should fix it.

@magesh1983
Copy link

Me too go into an issue when setting up LDAP authentication inside docker-zulip. Based upon little debugging found the following thing. Looks like entrypoint.sh: authenticationBackends() is not able to parse the ZULIP_AUTH_BACKENDS properly. It always sets only EmailAuthBackend irrespective of what is given in ZULIP_AUTH_BACKENDS. By the way I am able to get it work successfully using ZULIP_CUSTOM_SETTINGS: "AUTHENTICATION_BACKENDS = ('zproject.backends.EmailAuthBackend','zproject.backends.ZulipLDAPAuthBackend',)"

@Alexandre-Silva
Copy link

Hello,

Just letting you know that as of this writting, ZULIP_CUSTOM_SETTINGS works for me

@timabbott
Copy link
Member

Yeah, agreed, closing this as fixed through the array-format settings support we added a few years back.

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

No branches or pull requests

4 participants