perf(settings): Speed up InternetConnectivity setup check

Specify default protocol (https://) rather than let default handling test both http:// and https://

Signed-off-by: Josh <josh.t.richards@gmail.com>

Author: joshtrichards

apps/settings/lib/SetupChecks/InternetConnectivity.php

@@ -41,11 +41,12 @@ public function run(): SetupResult {
 		}
 
 		$siteArray = $this->config->getSystemValue('connectivity_check_domains', [
-			'www.nextcloud.com', 'www.startpage.com', 'www.eff.org', 'www.edri.org'
+			'https://www.nextcloud.com', 'https://www.startpage.com', 'https://www.eff.org', 'https://www.edri.org'
 		]);
 
 		foreach ($siteArray as $site) {
 			if ($this->isSiteReachable($site)) {
+				// successful as soon as one connection succeeds
 				return SetupResult::success();
 			}
 		}
@@ -55,19 +56,18 @@ public function run(): SetupResult {
 	/**
 	 * 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();
-			// if there is no protocol, test http:// AND https://
-			if (preg_match('/^https?:\/\//', $site) !== 1) {
-				$httpSite = 'http://' . $site . '/';
-				$client->get($httpSite);
-				$httpsSite = 'https://' . $site . '/';
-				$client->get($httpsSite);
-			} else {
-				$client->get($site);
-			}
+			$client->get($site);
 		} catch (\Exception $e) {
 			$this->logger->error('Cannot connect to: ' . $site, [
 				'app' => 'internet_connection_check',

config/config.sample.php

@@ -936,16 +936,16 @@
  *
  * Defaults to the following domains:
  *
- *  - www.nextcloud.com
- *  - www.startpage.com
- *  - www.eff.org
- *  - www.edri.org
+ *  - https://www.nextcloud.com
+ *  - https://www.startpage.com
+ *  - https://www.eff.org
+ *  - https://www.edri.org
  */
 'connectivity_check_domains' => [
-	'www.nextcloud.com',
-	'www.startpage.com',
-	'www.eff.org',
-	'www.edri.org'
+	'https://www.nextcloud.com',
+	'https://www.startpage.com',
+	'https://www.eff.org',
+	'https://www.edri.org'
 ],
 
 /**