chore: change NewAppServerFlag usage description

cli/internal/cli.go

@@ -169,7 +169,7 @@ var NewAppServerFlag = &cli.StringFlag{
 var NoDomainChecks bool
 var NoDomainChecksFlag = &cli.BoolFlag{
 	Name:        "no-domain-checks, D",
-	Usage:       "Disable app domain sanity checks",
+	Usage:       "Disable domain sanity checks",
 	Destination: &NoDomainChecks,
 }
 

cli/server/add.go

@@ -53,7 +53,7 @@ func cleanUp(domainName string) {
 // Docker manages SSH connection details. These are stored to disk in
 // ~/.docker. Abra can manage this completely for the user, so it's an
 // implementation detail.
-func newContext(c *cli.Context, domainName, username, port string) error {
+func newContext(c *cli.Context, name, host, username, port string) error {
 	store := contextPkg.NewDefaultDockerContextStore()
 	contexts, err := store.Store.List()
 	if err != nil {
@@ -61,15 +61,15 @@ func newContext(c *cli.Context, domainName, username, port string) error {
 	}
 
 	for _, context := range contexts {
-		if context.Name == domainName {
-			logrus.Debugf("context for %s already exists", domainName)
+		if context.Name == name {
+			logrus.Debugf("context for %s already exists", name)
 			return nil
 		}
 	}
 
-	logrus.Debugf("creating context with domain %s, username %s and port %s", domainName, username, port)
+	logrus.Debugf("creating context with name %s, host %s, username %s and port %s", name, host, username, port)
 
-	if err := client.CreateContext(domainName, username, port); err != nil {
+	if err := client.CreateContext(name, host, username, port); err != nil {
 		return err
 	}
 
@@ -112,11 +112,16 @@ Abra can then load SSH connection details from this configuratiion with:
 If "--local" is passed, then Abra assumes that the current local server is
 intended as the target server. This is useful when you want to have your entire
 Co-op Cloud config located on the server itself, and not on your local
-developer machine.
+developer machine. The domain is then set to "default".
+
+You can also pass "--no-domain-checks" flag to use any arbitrary name instead
+of a real domain. Host will be resolved with the "hostname" entry of your SSH
+configuration. Checks for a valid online domain will be skipped.
 `,
 	Flags: []cli.Flag{
 		internal.DebugFlag,
 		internal.NoInputFlag,
+		internal.NoDomainChecksFlag,
 		localFlag,
 	},
 	Before:    internal.SubCommandBefore,
@@ -150,23 +155,29 @@ developer machine.
 			return nil
 		}
 
-		if _, err := dns.EnsureIPv4(domainName); err != nil {
-			logrus.Fatal(err)
-		}
-
-		if err := createServerDir(domainName); err != nil {
-			logrus.Fatal(err)
+		if !internal.NoDomainChecks {
+			if _, err := dns.EnsureIPv4(domainName); err != nil {
+				logrus.Fatal(err)
+			}
 		}
 
 		hostConfig, err := sshPkg.GetHostConfig(domainName)
 		if err != nil {
 			logrus.Fatal(err)
 		}
+		if hostConfig.Host == "" {
+			hostConfig.Host = domainName
+		}
 
-		if err := newContext(c, domainName, hostConfig.User, hostConfig.Port); err != nil {
+		if err := createServerDir(domainName); err != nil {
 			logrus.Fatal(err)
 		}
 
+		if err := newContext(c, domainName, hostConfig.Host, hostConfig.User, hostConfig.Port); err != nil {
+			logrus.Fatal(err)
+			cleanUp(domainName)
+		}
+
 		logrus.Infof("attempting to create client for %s", domainName)
 		if _, err := client.New(domainName); err != nil {
 			cleanUp(domainName)

pkg/client/context.go

@@ -14,8 +14,7 @@ import (
 
 type Context = contextStore.Metadata
 
-func CreateContext(contextName string, user string, port string) error {
-	host := contextName
+func CreateContext(contextName string, host string, user string, port string) error {
 	if user != "" {
 		host = fmt.Sprintf("%s@%s", user, host)
 	}

pkg/config/env.go

@@ -16,25 +16,23 @@ import (
 	"github.com/sirupsen/logrus"
 )
 
-// getBaseDir retrieves the Abra base directory.
-func getBaseDir() string {
-	home := os.ExpandEnv("$HOME/.abra")
-	if customAbraDir, exists := os.LookupEnv("ABRA_DIR"); exists && customAbraDir != "" {
-		home = customAbraDir
+func getFromEnvOrDefault(envStr string, defaultStr string) string {
+	if envVar, exists := os.LookupEnv(envStr); exists && envVar != "" {
+		return envVar
 	}
-	return home
+	return defaultStr
 }
 
-var ABRA_DIR = getBaseDir()
+var ABRA_DIR = getFromEnvOrDefault("ABRA_DIR", os.ExpandEnv("$HOME/.abra"))
 var SERVERS_DIR = path.Join(ABRA_DIR, "servers")
 var RECIPES_DIR = path.Join(ABRA_DIR, "recipes")
 var VENDOR_DIR = path.Join(ABRA_DIR, "vendor")
 var BACKUP_DIR = path.Join(ABRA_DIR, "backups")
 var CATALOGUE_DIR = path.Join(ABRA_DIR, "catalogue")
 var RECIPES_JSON = path.Join(ABRA_DIR, "catalogue", "recipes.json")
-var REPOS_BASE_URL = "https://git.coopcloud.tech/coop-cloud"
-var CATALOGUE_JSON_REPO_NAME = "recipes-catalogue-json"
-var SSH_URL_TEMPLATE = "ssh://git@git.coopcloud.tech:2222/coop-cloud/%s.git"
+var REPOS_BASE_URL = getFromEnvOrDefault("ABRA_REPOS_BASE_URL", "https://git.coopcloud.tech/coop-cloud")
+var CATALOGUE_JSON_REPO_NAME = getFromEnvOrDefault("ABRA_CATALOGUE_JSON_REPO_NAME", "recipes-catalogue-json")
+var SSH_URL_TEMPLATE = getFromEnvOrDefault("ABRA_SSH_URL_TEMPLATE", "ssh://git@git.coopcloud.tech:2222/coop-cloud/%s.git")
 
 // envVarModifiers is a list of env var modifier strings. These are added to
 // env vars as comments and modify their processing by Abra, e.g. determining

pkg/dns/dns.go

@@ -9,7 +9,7 @@ import (
 func EnsureIPv4(domainName string) (string, error) {
 	ipv4, err := net.ResolveIPAddr("ip4", domainName)
 	if err != nil {
-		return "", err
+		return "", fmt.Errorf("unable to resolve ipv4 address for %s, %s", domainName, err)
 	}
 
 	// NOTE(d1): e.g. when there is only an ipv6 record available