Refactor port usage check

- Return value could be smarter so do again later

web/port_check.go

@@ -7,18 +7,18 @@ import (
 	"time"
 )
 
-func portCheck(address string, port int) (int, error) {
+func portCheck(address string, port int) (string, error) {
 	for {
 		conn, err := net.DialTimeout("tcp", fmt.Sprint(address, ":", port), time.Second)
 		if err != nil {
-			return port, nil
+			return fmt.Sprint(address, ":", port), nil
 		}
 		if conn != nil {
 			newPort := port + 1
-			log.Println(address, "is already in use.", fmt.Sprint(address, ":", newPort), "will use instead.")
+			log.Println(fmt.Sprint(address, ":", port), "is already in use.", fmt.Sprint(address, ":", newPort), "will use instead.")
 			port = newPort
 			continue
 		}
-		return port, nil
+		return fmt.Sprint(address, ":", port), nil
 	}
 }

web/web.go

@@ -99,17 +99,16 @@ func setupRoutes() *http.ServeMux {
 }
 
 func StartServer() {
-	port, err := portCheck(utils.LoadedConfig.ServerIp, utils.LoadedConfig.ServerPort)
-	address := fmt.Sprint(utils.LoadedConfig.ServerIp, ":", port)
-	l, err := net.Listen("tcp", address)
+	addr, err := portCheck(utils.LoadedConfig.ServerIp, utils.LoadedConfig.ServerPort)
+	l, err := net.Listen("tcp", addr)
 	if err != nil {
 		log.Fatalln(err)
 		return
 	}
 	mux := setupRoutes()
 	// There is no way to print the notification that the user can access after serving, so print it here instead
 	log.SetPrefix("[Web] ")
-	log.Printf("Start hosting on http://%s\n", address)
+	log.Printf("Start hosting on http://%s\n", addr)
 
 	if err = http.Serve(l, mux); err != nil {
 		log.SetPrefix("[Web] ")

web/web_d.go

@@ -110,17 +110,16 @@ func setupRoutes() *http.ServeMux {
 }
 
 func StartServer() {
-	port, err := portCheck(utils.LoadedConfig.ServerIp, utils.LoadedConfig.ServerPort)
-	address := fmt.Sprint(utils.LoadedConfig.ServerIp, ":", port)
-	l, err := net.Listen("tcp", address)
+	addr, err := portCheck(utils.LoadedConfig.ServerIp, utils.LoadedConfig.ServerPort)
+	l, err := net.Listen("tcp", addr)
 	if err != nil {
 		log.Fatalln(err)
 		return
 	}
 	mux := setupRoutes()
 	// There is no way to print the notification that the user can access after serving, so print it here instead
 	log.SetPrefix("[Web] ")
-	log.Printf("Start hosting on http://%s\n", address)
+	log.Printf("Start hosting on http://%s\n", addr)
 
 	if err = http.Serve(l, mux); err != nil {
 		log.SetPrefix("[Web] ")