According to the documentation, both `OBJECTSTORE_S3_SSL` and
`OBJECTSTORE_S3_AUTOCREATE` should default to `true`.
Currently, when these environment variables are not set, they default to
`false`. (See https://github.com/nextcloud/docker/issues/2308).
This fix works, because `strtolower(false)` returns the empty string. So
when `OBJECTSTORE_S3_SSL` is not set and `getenv('OBJECTSTORE_S3_SSL')`
returns `false`, the check `strtolower($use_ssl) !== 'false'` will
evaluate to `true`.
With this fix, both values will be `true` if they are
* not set
* the empty string
* any string that is not equal to `false` when converted to lowercase
This should now match the documented behavior.
Signed-off-by: Valentin Brandl <mail@vbrandl.net>
* Drop file_exists check for _FILE variables
A PHP warning is generated if the file does not exist.
Otherwise there is no indicator why the variable is not used.
Signed-off-by: J0WI <J0WI@users.noreply.github.com>
* fixup: typo in smtp.config.php
Signed-off-by: Josh <josh.t.richards@gmail.com>
---------
Signed-off-by: J0WI <J0WI@users.noreply.github.com>
Signed-off-by: Josh <josh.t.richards@gmail.com>
Co-authored-by: Josh <josh.t.richards@gmail.com>
Correctly set the redis password config from the REDIS_HOST_PASSWORD_FILE environment variable. Fix an issue similar to #1402 when using the REDIS_HOST_PASSWORD_FILE environment variable to provide the redis host secret.
Signed-off-by: Vesperia Art <vesperiaart@gmail.com>
The AWS S3 SSE-C key for server side encryption can now be set using
the environment variable `OBJECTSTORE_S3_SSE_C_KEY`.
Signed-off-by: Patrick Hobusch <patrick@hobusch.net>
The AWS S3 storage class can now be set using the environment variable
`OBJECTSTORE_S3_STORAGE_CLASS`.
The order of the AWS S3 environment variables in the `README.md` has
also been adjusted to be more "logical".
Signed-off-by: Patrick Hobusch <patrick@hobusch.net>
Web based updates aren't used in the image, but the button remains enabled and generates an error when clicked. Let's eliminate the web updater button.
Signed-off-by: Josh Richards <josh.t.richards@gmail.com>
We do not always need to configure variables as either a file or not for
all variables.
Instead, allow us to mix-match variable storage. This allows us to keep
all non-critical variables in regular (env) variables, but keep the
password in a (secret) file only location.
Signed-off-by: Olliver Schinagl <oliver@schinagl.nl>
* Added fetching autoconfig data from Docker secrets
Signed-off-by: Simon Tushev <tushev@users.noreply.github.com>
* Docker secrets now have priority over ENV variables; improved README.md
Signed-off-by: Simon Tushev <tushev@users.noreply.github.com>
* Disable rewrite ip for apache image
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* Run update.sh
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* Fix Redis Unix socket support
Rebase and rewrite the @epma01 pull request, based on last Nextcloud Docker changes
See: https://github.com/nextcloud/docker/pull/735
Signed-off-by: Tortue Torche <tortuetorche@users.noreply.github.com>
* Fix CI tests with update.sh script
Signed-off-by: Tortue Torche <tortuetorche@users.noreply.github.com>