Merge pull request #11193 from nextcloud/backport/11188/stable28

[stable28] fix(OCC): Fix error reporting and success message when adding a STUN …
nextcloud · Dec 11, 2023 · 0be6c4a · 0be6c4a
2 parents 0287398 + c3ecbb6
commit 0be6c4a
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/Command/Stun/Add.php b/lib/Command/Stun/Add.php
@@ -55,7 +55,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 		// check input, similar to stun-server.js
 		$host = parse_url($server, PHP_URL_HOST);
 		$port = parse_url($server, PHP_URL_PORT);
-		if (empty($host) || empty($host)) {
+		if (empty($host) || empty($port)) {
 			$output->writeln('<error>Incorrect value. Must be stunserver:port.</error>');
 			return 1;
 		}
@@ -70,7 +70,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 		$servers[] = "$host:$port";
 
 		$this->config->setAppValue('spreed', 'stun_servers', json_encode($servers));
-		$output->writeln('<info>Added ' . $server . '.</info>');
+		$output->writeln('<info>Added ' . "$host:$port" . '.</info>');
 		return 0;
 	}
 }