nextcloud / server

apps/settings/lib/SetupChecks/InternetConnectivity.php

Indentation +52 added lines, -52 removed lines

@@ -19,62 +19,62 @@
  * Checks if the server can connect to the internet using HTTPS and HTTP
  */
 class InternetConnectivity implements ISetupCheck {
-	public function __construct(
-		private IL10N $l10n,
-		private IConfig $config,
-		private IClientService $clientService,
-		private LoggerInterface $logger,
-	) {
-	}
+    public function __construct(
+        private IL10N $l10n,
+        private IConfig $config,
+        private IClientService $clientService,
+        private LoggerInterface $logger,
+    ) {
+    }
 
-	public function getCategory(): string {
-		return 'network';
-	}
+    public function getCategory(): string {
+        return 'network';
+    }
 
-	public function getName(): string {
-		return $this->l10n->t('Internet connectivity');
-	}
+    public function getName(): string {
+        return $this->l10n->t('Internet connectivity');
+    }
 
-	public function run(): SetupResult {
-		if ($this->config->getSystemValue('has_internet_connection', true) === false) {
-			return SetupResult::success($this->l10n->t('Internet connectivity is disabled in configuration file.'));
-		}
+    public function run(): SetupResult {
+        if ($this->config->getSystemValue('has_internet_connection', true) === false) {
+            return SetupResult::success($this->l10n->t('Internet connectivity is disabled in configuration file.'));
+        }
 
-		$siteArray = $this->config->getSystemValue('connectivity_check_domains', [
-			'https://connectivity.nextcloud.com', 'https://www.eff.org', 'https://edri.org'
-		]);
+        $siteArray = $this->config->getSystemValue('connectivity_check_domains', [
+            'https://connectivity.nextcloud.com', 'https://www.eff.org', 'https://edri.org'
+        ]);
 
-		foreach ($siteArray as $site) {
-			if ($this->isSiteReachable($site)) {
-				// successful as soon as one connection succeeds
-				return SetupResult::success();
-			}
-		}
-		return SetupResult::warning($this->l10n->t('This server has no working internet connection: Multiple endpoints could not be reached. This means that some of the features like mounting external storage, notifications about updates or installation of third-party apps will not work. Accessing files remotely and sending of notification emails might not work, either. Establish a connection from this server to the internet to enjoy all features.'));
-	}
+        foreach ($siteArray as $site) {
+            if ($this->isSiteReachable($site)) {
+                // successful as soon as one connection succeeds
+                return SetupResult::success();
+            }
+        }
+        return SetupResult::warning($this->l10n->t('This server has no working internet connection: Multiple endpoints could not be reached. This means that some of the features like mounting external storage, notifications about updates or installation of third-party apps will not work. Accessing files remotely and sending of notification emails might not work, either. Establish a connection from this server to the internet to enjoy all features.'));
+    }
 
-	/**
-	 * Checks if the Nextcloud server can connect to a specific URL
-	 * @param string $site site domain or full URL with http/https protocol
-	 * @return bool success/failure
-	 */
-	private function isSiteReachable(string $site): bool {
-		// if there is no protocol specified, test http:// first then, if necessary, https://
-		if (preg_match('/^https?:\/\//', $site) !== 1) {
-			$httpSite = 'http://' . $site . '/';
-			$httpsSite = 'https://' . $site . '/';
-			return $this->isSiteReachable($httpSite) || $this->isSiteReachable($httpsSite);
-		}
-		try {
-			$client = $this->clientService->newClient();
-			$client->head($site);
-		} catch (\Exception $e) {
-			$this->logger->error('Cannot connect to: ' . $site, [
-				'app' => 'internet_connection_check',
-				'exception' => $e,
-			]);
-			return false;
-		}
-		return true;
-	}
+    /**
+     * Checks if the Nextcloud server can connect to a specific URL
+     * @param string $site site domain or full URL with http/https protocol
+     * @return bool success/failure
+     */
+    private function isSiteReachable(string $site): bool {
+        // if there is no protocol specified, test http:// first then, if necessary, https://
+        if (preg_match('/^https?:\/\//', $site) !== 1) {
+            $httpSite = 'http://' . $site . '/';
+            $httpsSite = 'https://' . $site . '/';
+            return $this->isSiteReachable($httpSite) || $this->isSiteReachable($httpsSite);
+        }
+        try {
+            $client = $this->clientService->newClient();
+            $client->head($site);
+        } catch (\Exception $e) {
+            $this->logger->error('Cannot connect to: ' . $site, [
+                'app' => 'internet_connection_check',
+                'exception' => $e,
+            ]);
+            return false;
+        }
+        return true;
+    }
 }

Spacing +3 added lines, -3 removed lines

@@ -61,15 +61,15 @@
 	private function isSiteReachable(string $site): bool {
 		// if there is no protocol specified, test http:// first then, if necessary, https://
 		if (preg_match('/^https?:\/\//', $site) !== 1) {
-			$httpSite = 'http://' . $site . '/';
-			$httpsSite = 'https://' . $site . '/';
+			$httpSite = 'http://'.$site.'/';
+			$httpsSite = 'https://'.$site.'/';
 			return $this->isSiteReachable($httpSite) || $this->isSiteReachable($httpsSite);
 		}
 		try {
 			$client = $this->clientService->newClient();
 			$client->head($site);
 		} catch (\Exception $e) {
-			$this->logger->error('Cannot connect to: ' . $site, [
+			$this->logger->error('Cannot connect to: '.$site, [
 				'app' => 'internet_connection_check',
 				'exception' => $e,
 			]);