now the HTTP server is not started if HTTPS is used. Abstract a little the implementation, and remove useless print since r.Run is blocking

Author: umbynos

main.go

@@ -48,7 +48,6 @@ var (
 	version               = "x.x.x-dev" //don't modify it, Jenkins will take care
 	commit                = "xxxxxxxx"  //don't modify it, Jenkins will take care
 	port                  string
-	portSSL               string
 	requiredToolsAPILevel = "v1"
 )
 
@@ -196,7 +195,7 @@ func loop() {
 	log.SetLevel(log.InfoLevel)
 	log.SetOutput(os.Stdout)
 
-	// the important forlders of the agent
+	// the important folders of the agent
 	src, _ := os.Executable()
 	srcPath := paths.New(src)  // The path of the agent's binary
 	srcDir := srcPath.Parent() // The directory of the agent's binary
@@ -420,45 +419,33 @@ func loop() {
 	goa := v2.Server(agentDir.String())
 	r.Any("/v2/*path", gin.WrapH(goa))
 
-	go func() {
-		// check if certificates exist; if not, use plain http
-		if srcDir.Join("cert.pem").NotExist() {
-			log.Error("Could not find HTTPS certificate. Using plain HTTP only.")
-			return
-		}
+	// check if certificates exist; if not, use plain http
+	https := true
+	if srcDir.Join("cert.pem").NotExist() {
+		log.Error("Could not find HTTPS certificate. Using plain HTTP only.")
+		https = false
+	}
+	go startServer(r, https, srcDir)
+}
 
-		start := 8990
-		end := 9000
-		i := start
-		for i < end {
-			i = i + 1
-			portSSL = ":" + strconv.Itoa(i)
-			if err := r.RunTLS(*address+portSSL, srcDir.Join("cert.pem").String(), srcDir.Join("key.pem").String()); err != nil {
-				log.Printf("Error trying to bind to port: %v, so exiting...", err)
-				continue
-			} else {
-				log.Print("Starting server and websocket (SSL) on " + *address + "" + port)
-				break
-			}
+func startServer(r *gin.Engine, https bool, certDir *paths.Path) {
+	start := 8990
+	end := 9000
+	i := start
+	for i < end {
+		i = i + 1
+		port = ":" + strconv.Itoa(i)
+		var err error
+		if https {
+			err = r.RunTLS(*address+port, certDir.Join("cert.pem").String(), certDir.Join("key.pem").String())
+		} else {
+			err = r.Run(*address + port)
 		}
-	}()
-
-	go func() {
-		start := 8990
-		end := 9000
-		i := start
-		for i < end {
-			i = i + 1
-			port = ":" + strconv.Itoa(i)
-			if err := r.Run(*address + port); err != nil {
-				log.Printf("Error trying to bind to port: %v, so exiting...", err)
-				continue
-			} else {
-				log.Print("Starting server and websocket on " + *address + "" + port)
-				break
-			}
+		if err != nil {
+			log.Printf("Error trying to bind to port: %v, so exiting...", err)
+			continue
 		}
-	}()
+	}
 }
 
 func parseIni(filename string) (args []string, err error) {