Error creating New nextcloud instance

[epma01]

Caused by Compatibility with docker secrets (Fix #560) #729

+ echo 'New nextcloud instance'
New nextcloud instance
+ file_env NEXTCLOUD_ADMIN_PASSWORD
+ local 'var=NEXTCLOUD_ADMIN_PASSWORD'
+ local 'fileVar=NEXTCLOUD_ADMIN_PASSWORD_FILE'
+ local 'def='
./entrypoint.sh: line 171: syntax error: bad substitution

Introduced code is using bash indirect expansion but entrypoint.sh is not using bash and it is not supported

file_env() {
    local var="$1"
    local fileVar="${var}_FILE"
    local def="${2:-}"
>>>    if [ "${!var:-}" ] && [ "${!fileVar:-}" ]; then    <<< this line is causing the issue
        echo >&2 "error: both $var and $fileVar are set (but are exclusive)"
        exit 1
    fi

[tilosp]

cc @J0WI

[OhioVR]

When I try to instantiate the Nextcloud docker image from docker-compose I get an error "entrypoint.sh: 30: /entrypoint.sh: Bad substitution"

After that point it seems to fail to apply my environment variables. Looks like it should be fixed now however I was wondering if it were posted to docker hub at this time as I haven't seen the change work yet from my docker-compose pull.

[dassbj01]

Hi,
I experience the same issue ...
on top the linked git tree on docker hub is for sure NOT the one currently published ...
as the "changes" in the entrypoint.sh are NOT reflected.
That is a real blocker. as we are trying to get a zero touch nextcloud installation with configs and apps and therefor we heavily rely on a "stable" release of your image

[AEPKILL]

i have same issue, please update docker hub image.

[tilosp]

this is expected, the new image will be published to docker hub once docker-library/official-images#5834 is merged

[J0WI]

An update is scheduled, I'll investigate to fix the docker secrets feature.

[conanedogawa2]

Same probleme here.
Reproduce steps are as easy as :
docker pull nextcloud:latest
docker run -d -ti --name test nextcloud:latest
docker logs test

Thanks all for your work here.

[J0WI]

Containers are now rebuilt and I just added some tests to ensure that we catch such regressions before they are committed. Apologize for the inconvenience.