Backport #11446

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
nextcloud · Oct 11, 2018 · 4f983b7 · 4f983b7
1 parent 1f71412
commit 4f983b7
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/lib/private/Setup.php b/lib/private/Setup.php
@@ -438,16 +438,15 @@ public static function updateHtaccess() {
 		$config = \OC::$server->getSystemConfig();
 
 		// For CLI read the value from overwrite.cli.url
-		if(\OC::$CLI) {
+		if (\OC::$CLI) {
 			$webRoot = $config->getValue('overwrite.cli.url', '');
-			if($webRoot === '') {
+			if ($webRoot === '') {
 				return false;
 			}
-			$webRoot = parse_url($webRoot, PHP_URL_PATH);
-			if ($webRoot === null) {
+			if (!filter_var($webRoot, FILTER_VALIDATE_URL)) {
 				return false;
 			}
-			$webRoot = rtrim($webRoot, '/');
+			$webRoot = rtrim(parse_url($webRoot, PHP_URL_PATH), '/');
 		} else {
 			$webRoot = !empty(\OC::$WEBROOT) ? \OC::$WEBROOT : '/';
 		}