Skip to content
This repository was archived by the owner on Jan 3, 2019. It is now read-only.

Comments

Enable LDAP PHP extension in all versions#29

Closed
leoneparise wants to merge 2 commits intomasterfrom
unknown repository
Closed

Enable LDAP PHP extension in all versions#29
leoneparise wants to merge 2 commits intomasterfrom
unknown repository

Conversation

@leoneparise
Copy link

We think that most organizations have LDAP installed and should be enabled by default.

Fixes #24

@pierreozoux
Copy link
Contributor

@tianon @yosifkit what do you think?

I know the official images should stay small, but ldap is actually a broadly used feature.

Let's deal with this PR :)

@tianon
Copy link
Member

tianon commented Jan 22, 2016

Hmm, isn't this related to docker-library/php#85? Any idea what --with-ldap-sasl adds/changes?

@tianon
Copy link
Member

tianon commented Jan 22, 2016

FWIW, I do think that enabling this makes sense here (especially given that it appears to be a core feature of ownCloud), but I'd like to make sure we figure out what makes SASL support special before we move further (especially since that might affect what we do here).

@ItalyPaleAle
Copy link

+1 for LDAP support in the Docker image

@ItalyPaleAle
Copy link

@tianon SASL is another authentication method for OpenLDAP, but I wouldn't say it's particularly important. See: http://www.netid.washington.edu/documentation/ldapauth.aspx
I believe most users will use the "Simple" authentication, for example with a dedicated service account.

@sebastiangraf
Copy link

+1 for LDAP support in the Docker image. Like @EgoAleSum stated, I think as well that basic authentication is sufficient at the moment and would be a huge benefit for people using this image.

@maxnoe maxnoe mentioned this pull request Jul 11, 2016
@yosifkit yosifkit closed this in #74 Jul 11, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants