mirror of
https://github.com/lldap/lldap.git
synced 2026-04-25 16:25:55 +03:00
[GH-ISSUE #384] Support read-only /app filesystem #145
Labels
No labels
backend
blocked
bug
cleanup
dependencies
docker
documentation
duplicate
enhancement
enhancement
frontend
github_actions
good first issue
help wanted
help wanted
integration
invalid
ldap
pull-request
question
rust
rust
tests
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/lldap-lldap#145
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @RedlineTriad on GitHub (Nov 27, 2022).
Original GitHub issue: https://github.com/lldap/lldap/issues/384
I am unsure if this is feasible, but it would be nice to support read-only filesystems.
Currently, when starting lldap with a read-only filesystem, it will get to
Setup permissions..and then print a lot ofchownerrors:The issue can be reproduced with the following
docker-compose.ymlfile:Read only filesystems are very useful for security, reducing data loss when forgetting to mount data directories, and to reduce disk usage.
It would probably also be nice if it was the default in docker-compose examples.
@nitnelave commented on GitHub (Nov 27, 2022):
Right now, the main problem with a read-only
/appfolder is that the SQlite DB is in that folder... So we need to write to it.If you want to set up a read-only folder for the config and the static assets, you can do that by:
lldap run(potentially with a couple of options)I think with those 2 things, it should be enough. The entrypoint just sets up things by default for new users, for easier set up.
@RedlineTriad commented on GitHub (Nov 27, 2022):
I don't think that is correct?
The SQlite DB is stored in
/datanot/app, since I mount that, it is not an issue.@nitnelave commented on GitHub (Nov 27, 2022):
Oh, right. Then you just need to override the entrypoint when you start the
container, you don't need the script at all.
On Sun, 27 Nov 2022, 19:48 RedlineTriad, @.***> wrote: