Fix handling dev ssh key

githedgehog · Sep 30, 2023 · 6cc248c · 6cc248c
1 parent 4db4d5b
commit 6cc248c
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/pkg/fab/base.go b/pkg/fab/base.go
@@ -65,13 +65,18 @@ func (cfg *Base) Hydrate(preset cnc.Preset) error {
 		if !slices.Contains(cfg.AuthorizedKeys, val) {
 			cfg.AuthorizedKeys = append(cfg.AuthorizedKeys, val)
 		}
+		if cfg.Dev && !slices.Contains(cfg.AuthorizedKeys, DEV_SSH_KEY) {
+			cfg.AuthorizedKeys = append(cfg.AuthorizedKeys, DEV_SSH_KEY)
+		}
 	}
 
 	if preset == PRESET_VLAB {
 		cfg.Dev = true
 	}
 
-	slog.Warn("Attention! Development mode enabled - this is not secure! Default users and keys will be created.")
+	if cfg.Dev {
+		slog.Warn("Attention! Development mode enabled - this is not secure! Default users and keys will be created.")
+	}
 
 	return nil
 }

diff --git a/pkg/fab/ctrl_os_butane.tmpl.yaml b/pkg/fab/ctrl_os_butane.tmpl.yaml
@@ -6,7 +6,7 @@ passwd:
     - name: {{ .username }}
       ssh_authorized_keys:
         {{ range .authorizedKeys}}
-        - {{ . }}
+        - "{{ . }}"
         {{ end }}
       groups:
         - wheel

diff --git a/pkg/fab/fabric.go b/pkg/fab/fabric.go
@@ -122,7 +122,6 @@ func (cfg *Fabric) Build(basedir string, preset cnc.Preset, get cnc.GetComponent
 		slog.Info("Adding dev users", "users", users)
 		for idx := range users {
 			users[idx].SSHKeys = append(users[idx].SSHKeys, BaseConfig(get).AuthorizedKeys...)
-			users[idx].SSHKeys = append(users[idx].SSHKeys, DEV_SSH_KEY)
 			slog.Info("Adding dev ssh keys to user", "user", users[idx])
 		}
 	}