Added better error handling for the gRPC server.

 - Consumers of the CLI have a better exit code when it fails at startup
 - Logged message to stdout, so gRPC clients know, it is time to connect

Signed-off-by: Akos Kitta <[email protected]>

Author: Akos Kitta

Diff for: cli/daemon/daemon.go

@@ -16,14 +16,17 @@
 package daemon
 
 import (
+	"errors"
 	"fmt"
 	"io"
 	"io/ioutil"
 	"net"
 	"net/http"
 	"os"
 	"runtime"
+	"syscall"
 
+	"github.com/arduino/arduino-cli/cli/errorcodes"
 	"github.com/arduino/arduino-cli/cli/globals"
 	"github.com/arduino/arduino-cli/commands/daemon"
 	srv_commands "github.com/arduino/arduino-cli/rpc/commands"
@@ -87,8 +90,29 @@ func runDaemonCommand(cmd *cobra.Command, args []string) {
 	logrus.Infof("Starting daemon on TCP port %s", port)
 	lis, err := net.Listen("tcp", fmt.Sprintf(":%s", port))
 	if err != nil {
-		logrus.Fatalf("failed to listen: %v", err)
+		// Invalid port, such as "Foo"
+		var dnsError *net.DNSError
+		if errors.As(err, &dnsError) {
+			logrus.Error("Failed to listen on TCP port: %s. %v is unknown name.", port, dnsError.Name)
+			os.Exit(errorcodes.ErrCoreConfig)
+		}
+		// Invalid port number, such as -1
+		var addrError *net.AddrError
+		if errors.As(err, &addrError) {
+			logrus.Error("Failed to listen on TCP port: %s. %v is an invalid port.", port, addrError.Addr)
+			os.Exit(errorcodes.ErrCoreConfig)
+		}
+		// Port is already in use
+		var syscallErr *os.SyscallError
+		if errors.As(err, &syscallErr) && errors.Is(syscallErr.Err, syscall.EADDRINUSE) {
+			logrus.Error("Failed to listen on TCP port: %s. Address already in use.", port)
+			os.Exit(errorcodes.ErrNetwork)
+		}
+		logrus.Error("Failed to listen on TCP port: %s. Unexpected error: %v", port, err)
+		os.Exit(errorcodes.ErrGeneric)
 	}
+	// This message will show up on the stdout of the daemon process so that gRPC clients know it is time to connect.
+	logrus.Infof("Daemon is listening on TCP port %s...", port)
 	if err := s.Serve(lis); err != nil {
 		logrus.Fatalf("failed to serve: %v", err)
 	}