perf: only check https if http fails

Signed-off-by: Josh Richards <[email protected]>

apps/settings/lib/SetupChecks/InternetConnectivity.php

@@ -46,6 +46,7 @@ public function run(): SetupResult {
 
 		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',