update registrar to expect address instead of expecting domain

Author: Eslam-Nawara

node-registrar/Makefile

@@ -1,5 +1,5 @@
 run:
-	go run cmds/main.go --postgres-host localhost --postgres-port 5432 --postgres-db postgres --postgres-user postgres --postgres-password password --domain localhost --server-port 8080
+	go run cmds/main.go --postgres-host localhost --postgres-port 5432 --postgres-db postgres --postgres-user postgres --postgres-password password --address localhost --server-port 8080
 
 start-postgres:
 	docker run --name postgres -e POSTGRES_USER=postgres -e POSTGRES_PASSWORD=password -e POSTGRES_DB=postgres -p 5432:5432 -d postgres
@@ -14,7 +14,7 @@ server-start:
 	@go run cmds/main.go \
 		--server-port 8080 \
 		--debug \
-		--domain localhost \
+		--address  localhost \
 		--sql-log-level 4 \
 		--postgres-host localhost \
 		--postgres-port 5432 \

node-registrar/README.md

@@ -76,10 +76,10 @@ It offers operations like registring, listing, and updating farms and nodes, as
 Once the server is running, Swagger documentation can be accessed at:
 
 ```bash
-http://<domain>:<port>/swagger/index.html
+http://<address>:<port>/swagger/index.html
 ```
 
-Replace `<domain>` and `<port>` with the appropriate values.
+Replace `<address>` and `<port>` with the appropriate values.
 
 ## How to Use the Server
 
@@ -112,7 +112,7 @@ docker build -t registrar:latest -f node-registrar/Dockerfile .
    --max-open-conn=10 \
    --max-idle-conn=5 \
    --server-port=8080 \
-   --<domain=your-domain> \
+   --<address=your-address-or-domain> \
    --network=main\
    --admin_twin_id=1
    --debug

node-registrar/cmds/main.go

@@ -19,7 +19,7 @@ type flags struct {
 	db.Config
 	debug       bool
 	version     bool
-	domain      string
+	addr        string
 	serverPort  uint
 	network     string
 	adminTwinID uint64
@@ -53,7 +53,7 @@ func Run() error {
 	flag.BoolVar(&f.version, "v", false, "shows the package version")
 	flag.BoolVar(&f.debug, "debug", false, "allow debug logs")
 	flag.UintVar(&f.serverPort, "server-port", 8080, "server port")
-	flag.StringVar(&f.domain, "domain", "", "domain on which the server will be served")
+	flag.StringVar(&f.addr, "address", "", "address or domain on which the server will be served")
 	flag.StringVar(&f.network, "network", "dev", "the registrar network")
 	flag.Uint64Var(&f.adminTwinID, "admin-twin-id", 1, "admin twin ID")
 
@@ -91,7 +91,7 @@ func Run() error {
 
 	log.Info().Msgf("server is running on port :%d", f.serverPort)
 
-	err = s.Run(fmt.Sprintf("%s:%d", f.domain, f.serverPort))
+	err = s.Run(fmt.Sprintf("%s:%d", f.addr, f.serverPort))
 	if err != nil {
 		return errors.Wrap(err, "failed to run gin server")
 	}
@@ -104,19 +104,27 @@ func (f flags) validate() error {
 		return errors.Errorf("invalid port %d, server port should be in the valid port range 1–65535", f.serverPort)
 	}
 
-	if strings.TrimSpace(f.domain) == "" {
-		return errors.New("invalid domain name, domain name should not be empty")
-	}
-
 	if f.Config.SqlLogLevel < 1 || f.Config.SqlLogLevel > 4 {
 		return errors.Errorf("invalid sql log level %d, sql log level should be in the range 1-4", f.Config.SqlLogLevel)
 	}
 	if f.adminTwinID == 0 {
 		return errors.Errorf("invalid admin twin id %d, admin twin id should not be 0", f.adminTwinID)
 	}
 
-	if _, err := net.LookupHost(f.domain); err != nil {
-		return errors.Wrapf(err, "invalid domain %s", f.domain)
+	if strings.TrimSpace(f.addr) == "" {
+		return errors.New("invalid domain/address, should not be empty")
+	}
+
+	// Skip validation for common binding addresses
+	if f.addr == "0.0.0.0" || f.addr == "localhost" || f.addr == "127.0.0.1" {
+		return f.Config.Validate()
+	}
+
+	if net.ParseIP(f.addr) == nil {
+		// if not valid IP address, check if valid domain
+		if _, err := net.LookupHost(f.addr); err != nil {
+			return errors.Wrapf(err, "invalid domain %s", f.addr)
+		}
 	}
 
 	return f.Config.Validate()