-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
OBJECTSTORE_S3_SSL
does not work as documented
#2308
Labels
3. to review
bug
feature: auto config (environment variables)
Auto configuring via environment variables
Milestone
Comments
vbrandl
added a commit
to vbrandl/nextcloud-docker
that referenced
this issue
Oct 8, 2024
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 nextcloud#2308). 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.
vbrandl
added a commit
to vbrandl/nextcloud-docker
that referenced
this issue
Oct 8, 2024
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 nextcloud#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.
This comment has been minimized.
This comment has been minimized.
joshtrichards
added
bug
feature: auto config (environment variables)
Auto configuring via environment variables
2. developing
3. to review
and removed
2. developing
labels
Oct 9, 2024
vbrandl
added a commit
to vbrandl/nextcloud-docker
that referenced
this issue
Oct 9, 2024
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 nextcloud#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 <[email protected]>
J0WI
pushed a commit
that referenced
this issue
Oct 24, 2024
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 #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 <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
3. to review
bug
feature: auto config (environment variables)
Auto configuring via environment variables
The documentation for
OBJECTSTORE_S3_SSL
states that it defaults totrue
:I would assume, not setting
OBJECTSTORE_S3_SSL
at all would cause Nextcloud to connect to my S3 bucket via HTTPS.I use the following
compose.yml
for testing.If i start a new instance like this, I get the following
config.php
:use_ssl
is set tofalse
but I would expect it to betrue
.Now I delete
./data/nextcloud
, uncomment- OBJECTSTORE_S3_SSL=true
incompose.yml
and start the compose project again anduse_ssl
inconfig.php
will be set totrue
.So the documentation does not match how the image actually behaves.
The text was updated successfully, but these errors were encountered: