Ping more privacy respecting organizations

Signed-off-by: Roeland Jago Douma <[email protected]>
nextcloud · Apr 5, 2018 · 176b4d6 · 176b4d6
1 parent feef3cb
commit 176b4d6
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 7 deletions.
diff --git a/settings/Controller/CheckSetupController.php b/settings/Controller/CheckSetupController.php
@@ -104,8 +104,10 @@ private function isInternetConnectionWorking() {
 		}
 
 		$siteArray = ['www.nextcloud.com',
-						'www.google.com',
-						'www.github.com'];
+						'www.startpage.com',
+						'www.eff.org',
+						'www.edri.org',
+			];
 
 		foreach($siteArray as $site) {
 			if ($this->isSiteReachable($site)) {
@@ -116,7 +118,7 @@ private function isInternetConnectionWorking() {
 	}
 
 	/**
-	* Chceks if the ownCloud server can connect to a specific URL using both HTTPS and HTTP
+	* Checks if the Nextcloud server can connect to a specific URL using both HTTPS and HTTP
 	* @return bool
 	*/
 	private function isSiteReachable($sitename) {

diff --git a/tests/Settings/Controller/CheckSetupControllerTest.php b/tests/Settings/Controller/CheckSetupControllerTest.php
@@ -150,7 +150,7 @@ public function testIsInternetConnectionFail() {
 			->method('get')
 			->will($this->throwException(new \Exception()));
 
-		$this->clientService->expects($this->exactly(3))
+		$this->clientService->expects($this->exactly(4))
 			->method('newClient')
 			->will($this->returnValue($client));
 
@@ -285,13 +285,17 @@ public function testCheck() {
 			->will($this->throwException(new \Exception()));
 		$client->expects($this->at(1))
 			->method('get')
-			->with('http://www.google.com/', [])
+			->with('http://www.startpage.com/', [])
 			->will($this->throwException(new \Exception()));
 		$client->expects($this->at(2))
 			->method('get')
-			->with('http://www.github.com/', [])
+			->with('http://www.eff.org/', [])
 			->will($this->throwException(new \Exception()));
-		$this->clientService->expects($this->exactly(3))
+		$client->expects($this->at(3))
+			->method('get')
+			->with('http://www.edri.org/', [])
+			->will($this->throwException(new \Exception()));
+		$this->clientService->expects($this->exactly(4))
 			->method('newClient')
 			->will($this->returnValue($client));
 		$this->urlGenerator->expects($this->at(0))