Use new findport fonction (#6)

scripts/_common.sh

@@ -102,20 +102,6 @@ CHECK_VAR () {	# Vérifie que la variable n'est pas vide.
 	test -n "$1" || (echo "$2" >&2 && false)
 }
 
-# Ignore the yunohost-cli log to prevent errors with conditionals commands
-# usage: NO_LOG COMMAND
-# Simply duplicate the log, execute the yunohost command and replace the log without the result of this command
-# It's a very badly hack...
-# Petite copie perso à mon usage ;)
-NO_LOG() {
-  ynh_cli_log=/var/log/yunohost/yunohost-cli.log
-  sudo cp -a ${ynh_cli_log} ${ynh_cli_log}-move
-  eval $@
-  exit_code=$?
-  sudo mv ${ynh_cli_log}-move ${ynh_cli_log}
-  return $?
-}
-
 CHECK_PATH () {	# Vérifie la présence du / en début de path. Et son absence à la fin.
 	if [ "${path:0:1}" != "/" ]; then    # Si le premier caractère n'est pas un /
 		path="/$path"    # Ajoute un / en début de path
@@ -138,16 +124,22 @@ CHECK_FINALPATH () {	# Vérifie que le dossier de destination n'est pas déjà u
 	fi
 }
 
-FIND_PORT () {	# Cherche un port libre.
+# Find a free port and return it
-# $1 = Numéro de port pour débuter la recherche.
+#
+# example: port=$(ynh_find_port 8080)
+#
+# usage: ynh_find_port begin_port
+# | arg: begin_port - port to start to search
+ynh_find_port () {
 	port=$1
-	while ! sudo yunohost app checkport $port ; do
+	test -n "$port" || ynh_die "The argument of ynh_find_port must be a valid port."
-		port=$((port+1))
+	while netcat -z 127.0.0.1 $port       # Check if the port is free
+	do
+		port=$((port+1))	# Else, pass to next port
 	done
-	CHECK_VAR "$port" "port empty"
+	echo $port
 }
 
-
 ### REMOVE SCRIPT
 
 REMOVE_NGINX_CONF () {	# Suppression de la configuration nginx

scripts/install

@@ -16,12 +16,10 @@ CHECK_DOMAINPATH	# Vérifie la disponibilité du path et du domaine.
 CHECK_FINALPATH	# Vérifie que le dossier de destination n'est pas déjà utilisé.
 
 # Ouvre le port dans le firewall
-NO_LOG FIND_PORT 8448	# Cherche un port libre.
+synapse_tls_port=$(ynh_find_port 8448)
-synapse_tls_port=$port
+synapse_port=$(ynh_find_port 8008)
-NO_LOG FIND_PORT 8008	# Cherche un port libre.
+turnserver_tls_port=$(ynh_find_port 5349)
-synapse_port=$port
+
-NO_LOG FIND_PORT 5349	# Cherche un port libre.
-turnserver_tls_port=$port
 sudo yunohost firewall allow --no-upnp TCP $synapse_tls_port > /dev/null 2>&1
 sudo yunohost firewall allow --no-upnp Both $turnserver_tls_port > /dev/null 2>&1
 

scripts/restore

@@ -111,20 +111,6 @@ CHECK_VAR () {	# Vérifie que la variable n'est pas vide.
 	test -n "$1" || (echo "$2" >&2 && false)
 }
 
-# Ignore the yunohost-cli log to prevent errors with conditionals commands
-# usage: NO_LOG COMMAND
-# Simply duplicate the log, execute the yunohost command and replace the log without the result of this command
-# It's a very badly hack...
-# Petite copie perso à mon usage ;)
-NO_LOG() {
-  ynh_cli_log=/var/log/yunohost/yunohost-cli.log
-  sudo cp -a ${ynh_cli_log} ${ynh_cli_log}-move
-  eval $@
-  exit_code=$?
-  sudo mv ${ynh_cli_log}-move ${ynh_cli_log}
-  return $?
-}
-
 CHECK_PATH () {	# Vérifie la présence du / en début de path. Et son absence à la fin.
 	if [ "${path:0:1}" != "/" ]; then    # Si le premier caractère n'est pas un /
 		path="/$path"    # Ajoute un / en début de path
@@ -147,16 +133,6 @@ CHECK_FINALPATH () {	# Vérifie que le dossier de destination n'est pas déjà u
 	fi
 }
 
-FIND_PORT () {	# Cherche un port libre.
-# $1 = Numéro de port pour débuter la recherche.
-	port=$1
-	while ! sudo yunohost app checkport $port ; do
-		port=$((port+1))
-	done
-	CHECK_VAR "$port" "port empty"
-}
-
-
 ### REMOVE SCRIPT
 
 REMOVE_NGINX_CONF () {	# Suppression de la configuration nginx