Validate the website field input to be a valid URL

Signed-off-by: Joas Schilling <[email protected]>

apps/settings/lib/Controller/UsersController.php

@@ -491,6 +491,9 @@ protected function saveUserSettings(IUser $user, array $data): array {
 			if ($e->getMessage() === IAccountManager::PROPERTY_PHONE) {
 				throw new \InvalidArgumentException($this->l10n->t('Unable to set invalid phone number'));
 			}
+			if ($e->getMessage() === IAccountManager::PROPERTY_WEBSITE) {
+				throw new \InvalidArgumentException($this->l10n->t('Unable to set invalid website'));
+			}
 			throw new \InvalidArgumentException($this->l10n->t('Some account data was invalid'));
 		}
 	}

lib/private/Accounts/AccountManager.php

@@ -120,6 +120,25 @@ protected function parsePhoneNumber(string $input): string {
 		throw new \InvalidArgumentException(self::PROPERTY_PHONE);
 	}
 
+	/**
+	 *
+	 * @param string $input
+	 * @return string
+	 * @throws \InvalidArgumentException When the website did not have http(s) as protocol or the host name was empty
+	 */
+	protected function parseWebsite(string $input): string {
+		$parts = parse_url($input);
+		if (!isset($parts['scheme']) || ($parts['scheme'] !== 'https' && $parts['scheme'] !== 'http')) {
+			throw new \InvalidArgumentException(self::PROPERTY_WEBSITE);
+		}
+
+		if (!isset($parts['host']) || $parts['host'] === '') {
+			throw new \InvalidArgumentException(self::PROPERTY_WEBSITE);
+		}
+
+		return $input;
+	}
+
 	/**
 	 * update user record
 	 *
@@ -155,6 +174,17 @@ public function updateUser(IUser $user, array $data, bool $throwOnData = false):
 			}
 		}
 
+		if (isset($data[self::PROPERTY_WEBSITE]) && $data[self::PROPERTY_WEBSITE]['value'] !== '') {
+			try {
+				$data[self::PROPERTY_WEBSITE]['value'] = $this->parseWebsite($data[self::PROPERTY_WEBSITE]['value']);
+			} catch (\InvalidArgumentException $e) {
+				if ($throwOnData) {
+					throw $e;
+				}
+				$data[self::PROPERTY_WEBSITE]['value'] = '';
+			}
+		}
+
 		$allowedScopes = [
 			self::SCOPE_PRIVATE,
 			self::SCOPE_LOCAL,

tests/lib/Accounts/AccountManagerTest.php

@@ -455,4 +455,30 @@ public function testParsePhoneNumber(string $phoneInput, string $defaultRegion,
 			self::assertEquals($phoneNumber, self::invokePrivate($instance, 'parsePhoneNumber', [$phoneInput]));
 		}
 	}
+
+	public function dataParseWebsite(): array {
+		return [
+			['https://nextcloud.com', 'https://nextcloud.com'],
+			['http://nextcloud.com', 'http://nextcloud.com'],
+			['ftp://nextcloud.com', null],
+			['//nextcloud.com/', null],
+			['https:///?query', null],
+		];
+	}
+
+	/**
+	 * @dataProvider dataParseWebsite
+	 * @param string $websiteInput
+	 * @param string|null $websiteOutput
+	 */
+	public function testParseWebsite(string $websiteInput, ?string $websiteOutput): void {
+		$instance = $this->getInstance();
+
+		if ($websiteOutput === null) {
+			$this->expectException(\InvalidArgumentException::class);
+			self::invokePrivate($instance, 'parseWebsite', [$websiteInput]);
+		} else {
+			self::assertEquals($websiteOutput, self::invokePrivate($instance, 'parseWebsite', [$websiteInput]));
+		}
+	}
 }