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>
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>
