Skip to content

Conversation

@J0WI
Copy link
Contributor

@J0WI J0WI commented Sep 1, 2022

@J0WI J0WI merged commit 19256cd into nextcloud:master Sep 6, 2022
@J0WI J0WI deleted the uid branch September 6, 2022 16:46
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Sep 6, 2022
Changes:

- https///github.com/nextcloud/docker/commit/b30812d: Runs update.sh
- https///github.com/nextcloud/docker/commit/19256cd: Allow to run with custom uid (https///github.com/nextcloud/docker/pull/1812)
- https///github.com/nextcloud/docker/commit/2a5bc6f: enable mariadb auto upgrade (https///github.com/nextcloud/docker/pull/1814)
- https///github.com/nextcloud/docker/commit/338b483: Runs update.sh
- https///github.com/nextcloud/docker/commit/d447c57: fix SMTP_PASSWORD_FILE not triggering auth activation (https///github.com/nextcloud/docker/pull/1614)
- https///github.com/nextcloud/docker/commit/685803d: Remove 22 (https///github.com/nextcloud/docker/pull/1813)
@PrivatePuffin
Copy link

Afaik the redis section also needs patching @J0WI

@J0WI
Copy link
Contributor Author

J0WI commented Sep 7, 2022

You can either run the image as limited user or change the system config. But it makes no sense if the system config is world writable. Probably needs a hint documentations.

@namelivia
Copy link

Hello! I really appreciate this update, could you please explain briefly how to use it? I'm not super sure while reading the code. Should I just pass uid and gid as env variables to the container?

@J0WI
Copy link
Contributor Author

J0WI commented Oct 9, 2022

You should run the container with the --user option.

@Chuckame
Copy link

Chuckame commented Feb 9, 2023

Hello, this is not working with a fresh setup with apache, because of the redis stuff just after (permission denied during writing redis client params).
I it a problem if we chown $user:$group the folder /usr/local/etc/php/conf.d before ?

@J0WI
Copy link
Contributor Author

J0WI commented Feb 17, 2023

I it a problem if we chown $user:$group the folder /usr/local/etc/php/conf.d before ?

see #763

ananace pushed a commit to ananace/docker-nextcloud that referenced this pull request May 10, 2024
@g0r3
Copy link

g0r3 commented Dec 30, 2024

Is this also supposed to work with Nexcloud AIO? I want to remap the www-data user to 1000 and tried using it with docker run -d --user=1000 ... but all I get is a container failed to start.

@joshtrichards
Copy link
Member

@g0r3 This repo/image set has nothing to do with AIO.

@goooooouwa
Copy link

@joshtrichards is there a possibility to also port this change to AIO? AIO's nextcloud docker entrypoint.sh seems like sharing a lot of the same code.

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.

Allow the UID/GID to be changed

8 participants