Merge pull request #7485 from nextcloud/fix_7001

Also cache invalid DiscoveryService response

Author: rullzer

apps/files_sharing/lib/External/Storage.php

@@ -280,7 +280,7 @@ private function testRemoteUrl($url) {
 			$returnValue = false;
 		}
 
-		$cache->set($url, $returnValue);
+		$cache->set($url, $returnValue, 60*60*24);
 		return $returnValue;
 	}
 

lib/private/OCS/DiscoveryService.php

@@ -81,11 +81,10 @@ public function discover($remote, $service) {
 			}
 		} catch (\Exception $e) {
 			// if we couldn't discover the service or any end-points we return a empty array
-			return [];
 		}
 
 		// Write into cache
-		$this->cache->set($remote . '#' . $service, json_encode($discoveredServices));
+		$this->cache->set($remote . '#' . $service, json_encode($discoveredServices), 60*60*24);
 		return $discoveredServices;
 	}