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

Ensure files are owned by the openldap user. #561

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bglimepoint
Copy link

@bglimepoint bglimepoint commented May 14, 2021

Thank you for all your work on this image, it's really useful.

We were using this image and found some stray files are created with a 0:0 uid:gid, rather than the openldap user that is configured.

This pr just adds some chowns to fix those permissions :-)

By default these are created as root (the current user) so chown them to
openldap to respect the configured uid:gid.

```
$ find slapd.d -uid 0 -o -gid 0
slapd.d/docker-openldap-was-admin-password-set
slapd.d/docker-openldap-was-started-with-tls
```

After this change these are owned by the intended uid:gid.
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.

1 participant