search

linuxserver / docker-dokuwiki

GNU General Public License v3.0
113 stars 24 forks source link

Add php7-ldap package to support LDAP authentication #7

Closed lorenzschmid closed 4 years ago

lorenzschmid commented 4 years ago

Adds the installation of the php7-ldap package to the Dockerfiles in order to support LDAP authentication.

Description:

Adds the installation of the php7-ldap package to all three Dockerfiles in order to support LDAP authentication.

Benefits of this PR and context:

Dokuwiki comes pre-installed with a plugin that allows authentication via LDAP. In order for the LDAP plugin to work the php7-ldap package is required on a system level.

How Has This Been Tested?

I didn't build the docker image (not a docker expert) but simply SSHed into the container and executed the very same command to install the missing package.

Source / References:

Closes #6

LinuxServer-CI commented 4 years ago

I am a bot, here are the test results for this PR: https://lsio-ci.ams3.digitaloceanspaces.com/lspipepr/dokuwiki/2018-04-22b-pkg-6f719813-pr-7/index.html https://lsio-ci.ams3.digitaloceanspaces.com/lspipepr/dokuwiki/2018-04-22b-pkg-6f719813-pr-7/shellcheck-result.xml

lorenzschmid commented 4 years ago

Thanks for the feedback. I updated my commit/PR.

LinuxServer-CI commented 4 years ago

I am a bot, here are the test results for this PR: https://lsio-ci.ams3.digitaloceanspaces.com/lspipepr/dokuwiki/2018-04-22b-pkg-064a2ddb-pr-7/index.html https://lsio-ci.ams3.digitaloceanspaces.com/lspipepr/dokuwiki/2018-04-22b-pkg-064a2ddb-pr-7/shellcheck-result.xml