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

LDAP_ADMIN_PASSWORD with space breaks container setup #167

Closed
wijata opened this issue Nov 30, 2017 · 3 comments
Closed

LDAP_ADMIN_PASSWORD with space breaks container setup #167

wijata opened this issue Nov 30, 2017 · 3 comments

Comments

@wijata
Copy link

wijata commented Nov 30, 2017

just try setting up
LDAP_ADMIN_PASSWORD to "foo bar" value
probbaly same for LDAP_CONFIG_PASSWORD

@goeland86
Copy link

goeland86 commented Jan 5, 2018

What exactly did you use as your command when setting it up? I'd have expected that if you put the quotes around the password it would be set with the space taken into account. Like so:
docker run --env LDAP_ORGANISATION="My Company" --env LDAP_DOMAIN="my-company.com" \ --env LDAP_ADMIN_PASSWORD="Jon Sn0w" --detach osixia/openldap:1.1.11

@wijata
Copy link
Author

wijata commented Jan 5, 2018

I actually left the project, but as I remember there are following codes in startup.sh file, try replacing password with two words
slappasswd -s $LDAP_CONFIG_PASSWORD
ldapmodify -h localhost -p 389 -D cn=admin,$LDAP_BASE_DN -w $LDAP_ADMIN_PASSWORD -f $LDIF_FILE

@goeland86
Copy link

I'd guess it's a small bug related to improper escaping of the password variable. It should have the quotes around it, like so:
ldapmodify -h localhost -p 389 -D cn=admin,"$LDAP_BASE_DN" -w "$LDAP_ADMIN_PASSWORD" -f "$LDIF_FILE"

Tricky little bash idiosyncracies...

hideyukiogino added a commit to ramencloud/docker-fusiondirectory-openldap that referenced this issue Jul 10, 2019
- Update the version of `osixia/openldap` from 1.1.10 to 1.2.4 to use
  functionalites avaiable in the 1.2.4.

- Delete `startup.sh.diff` which was a workaround for the issue 167 of
  `osixia/openldap`, since the issue is fixed in 1.2.4 of
  `osixia/openldap`, and the workaround is not needed anymore.
  osixia/docker-openldap#167

- Remove lines which setup multi-master replication from init.sh,
  since those setup is done in `osixia/openldap`.

- Change the repository for fusiondirectory pakcages in order to fix
  build issue. Fusiondirectory pakcages of v1.2-1 is not available in
  http://repos.fusiondirectory.org/fusiondirectory-current/debian-jessie
  anymore. Instead of that, we need to use the following repository:
  http://repos.fusiondirectory.org/fusiondirectory-releases/fusiondirectory-1.2/debian-jessie
hideyukiogino added a commit to ramencloud/docker-fusiondirectory-openldap that referenced this issue Jul 10, 2019
- Update the version of `osixia/openldap` from 1.1.10 to 1.2.4 to use
  functionalites avaiable in the 1.2.4.

- Delete `startup.sh.diff` which was a workaround for the issue 167 of
  `osixia/openldap`, since the issue is fixed in 1.2.4 of
  `osixia/openldap`, and the workaround is not needed anymore.
  osixia/docker-openldap#167

- Remove lines which setup multi-master replication from init.sh,
  since those setup is done in `osixia/openldap`.

- Change the repository for fusiondirectory pakcages in order to fix
  build issue. Fusiondirectory pakcages of v1.2-1 is not available in
  http://repos.fusiondirectory.org/fusiondirectory-current/debian-jessie
  anymore. Instead of that, we need to use the following repository:
  http://repos.fusiondirectory.org/fusiondirectory-releases/fusiondirectory-1.2/debian-jessie
hideyukiogino added a commit to ramencloud/docker-fusiondirectory-openldap that referenced this issue Jul 10, 2019
)

- Update the version of `osixia/openldap` from 1.1.10 to 1.2.4 to use
  functionalites avaiable in the 1.2.4.

- Delete `startup.sh.diff` which was a workaround for the issue 167 of
  `osixia/openldap`, since the issue is fixed in 1.2.4 of
  `osixia/openldap`, and the workaround is not needed anymore.
  osixia/docker-openldap#167

- Remove lines which setup multi-master replication from init.sh,
  since those setup is done in `osixia/openldap`.

- Change the repository for fusiondirectory pakcages in order to fix
  build issue. Fusiondirectory pakcages of v1.2-1 is not available in
  http://repos.fusiondirectory.org/fusiondirectory-current/debian-jessie
  anymore. Instead of that, we need to use the following repository:
  http://repos.fusiondirectory.org/fusiondirectory-releases/fusiondirectory-1.2/debian-jessie
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants