Merge pull request #5117 from nextcloud/enh/noid/improve-config

nextcloud: config files: do not compare against false
nextcloud · Aug 12, 2024 · 6986ecc · 6986ecc
2 parents c369b1f + 151b05e
commit 6986ecc
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/Containers/nextcloud/config/apps.config.php b/Containers/nextcloud/config/apps.config.php
@@ -13,6 +13,6 @@
       ),
   ),
 );
-if (getenv('APPS_ALLOWLIST') !== false) {
+if (getenv('APPS_ALLOWLIST')) {
     $CONFIG['appsallowlist'] = explode(" ", getenv('APPS_ALLOWLIST'));
 }
diff --git a/Containers/nextcloud/config/proxy.config.php b/Containers/nextcloud/config/proxy.config.php
@@ -1,13 +1,13 @@
 <?php
-if (getenv('HTTP_PROXY') !== false) {
+if (getenv('HTTP_PROXY')) {
     $CONFIG['proxy'] = getenv('HTTP_PROXY');
 }
-if (getenv('HTTPS_PROXY') !== false) {
+if (getenv('HTTPS_PROXY')) {
     $CONFIG['proxy'] = getenv('HTTPS_PROXY');
 }
-if (getenv('PROXY_USER_PASSWORD') !== false) {
+if (getenv('PROXY_USER_PASSWORD')) {
     $CONFIG['proxyuserpwd'] = getenv('PROXY_USER_PASSWORD');
 }
-if (getenv('NO_PROXY') !== false) {
+if (getenv('NO_PROXY')) {
     $CONFIG['proxyexclude'] = explode(',', getenv('NO_PROXY'));
 }
diff --git a/Containers/nextcloud/config/redis.config.php b/Containers/nextcloud/config/redis.config.php
@@ -9,13 +9,13 @@
     ),
   );
 
-  if (getenv('REDIS_HOST_PORT') !== false) {
+  if (getenv('REDIS_HOST_PORT')) {
     $CONFIG['redis']['port'] = (int) getenv('REDIS_HOST_PORT');
   } elseif (getenv('REDIS_HOST')[0] != '/') {
     $CONFIG['redis']['port'] = 6379;
   }
 
-  if (getenv('REDIS_DB_INDEX') !== false) {
+  if (getenv('REDIS_DB_INDEX')) {
     $CONFIG['redis']['dbindex'] = (int) getenv('REDIS_DB_INDEX');
   }
 }