-
Notifications
You must be signed in to change notification settings - Fork 163
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
Add environments PUID/PGID of /var/lib/dav
folder.
#28
base: master
Are you sure you want to change the base?
Conversation
This allows the user to set the default umask / chmod of files, which are uploaded through an WebDAV client
Add optional umask to docker-entrypoint.sh
This looks good to me. This PR would open up a lot of possibilities (mounting system directories, particularly VM shared folders, etc). Can a contributor take a look or approve / merge as well? |
@@ -48,6 +48,8 @@ services: | |||
AUTH_TYPE: Digest | |||
USERNAME: alice | |||
PASSWORD: secret1234 | |||
PUID: 1000 | |||
GUID: 1000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggestion: A good idea to add PUMASK
in the compose example here. ex PUMASK: ???
@@ -104,4 +106,7 @@ All environment variables are optional. You probably want to at least specify `U | |||
* **`PASSWORD`**: Authenticate with this password (and the username above). This is ignored if you bind mount your own authentication file to `/user.passwd`. | |||
* **`ANONYMOUS_METHODS`**: Comma-separated list of HTTP request methods (eg, `GET,POST,OPTIONS,PROPFIND`). Clients can use any method you specify here without authentication. Set to `ALL` to disable authentication. The default is to disallow any anonymous access. | |||
* **`SSL_CERT`**: Set to `selfsigned` to generate a self-signed certificate and enable Apache's SSL module. If you specify `SERVER_NAMES`, the first domain is set as the Common Name. | |||
* **`PUID`**: file owner's UID of `/var/lib/dav` | |||
* **`PGID`**: file owner's GID of `/var/lib/dav` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggestion: Likewise here add the description for PUMASK
. ex PUMASK: umask of /var/lib/dav/data
Signed-off-by: Jadin Heaston <jadin.heaston@como.gov>
Hi there, will this PR be merged some day? :D @tkodev |
a easy way to change the data folder's owner, Fix issues #24 #27