Fix init deleting device ID immediately after generating it; fix sync failure for non-manual syncs

This commit is contained in:
2024-05-30 23:25:08 -04:00
parent c3dd0ae737
commit 55a8f0e9ca
2 changed files with 18 additions and 16 deletions
+13 -13
View File
@@ -10,12 +10,10 @@ import (
) )
// TempInit ensures libmutton directories exist and writes the libmutton configuration file // TempInit ensures libmutton directories exist and writes the libmutton configuration file
func TempInit(configFileMap map[string]string) { // TODO if run in append mode, extend old config file with new values, rather than creating from scratch
func TempInit(configFileMap map[string]string, append bool) {
// create EntryRoot and ConfigDir // create EntryRoot and ConfigDir
DirInit() DirInit(append)
// remove existing config file
removeFile(ConfigPath)
if configFileMap["textEditor"] == "" { if configFileMap["textEditor"] == "" {
configFileMap["textEditor"] = textEditorFallback() configFileMap["textEditor"] = textEditorFallback()
@@ -68,7 +66,7 @@ func GpgKeyGen() string {
} }
// DirInit creates the libmutton directories // DirInit creates the libmutton directories
func DirInit() { func DirInit(preserveOldConfigDir bool) {
// create EntryRoot // create EntryRoot
err := os.MkdirAll(EntryRoot, 0700) err := os.MkdirAll(EntryRoot, 0700)
if err != nil { if err != nil {
@@ -76,14 +74,16 @@ func DirInit() {
os.Exit(1) os.Exit(1)
} }
// remove existing config directory (if it exists) // remove existing config directory (if it exists and not in append mode)
_, isAccessible := TargetIsFile(ConfigDir, false, 1) if !preserveOldConfigDir {
if isAccessible { _, isAccessible := TargetIsFile(ConfigDir, false, 1)
err = os.RemoveAll(ConfigDir) if isAccessible {
if err != nil { err = os.RemoveAll(ConfigDir)
fmt.Println(AnsiError + "Failed to remove existing config directory: " + err.Error() + AnsiReset) if err != nil {
os.Exit(1) fmt.Println(AnsiError + "Failed to remove existing config directory: " + err.Error() + AnsiReset)
os.Exit(1)
}
} }
} }
+5 -3
View File
@@ -24,11 +24,13 @@ const (
func getSSHClient(manualSync bool) (*ssh.Client, string, bool) { func getSSHClient(manualSync bool) (*ssh.Client, string, bool) {
// get SSH config info, exit if not configured (displaying an error if the sync job was called manually) // get SSH config info, exit if not configured (displaying an error if the sync job was called manually)
var sshUserConfig []string var sshUserConfig []string
var missingValueError string
if manualSync { if manualSync {
sshUserConfig = backend.ReadConfig([]string{"sshUser", "sshIP", "sshPort", "sshKey", "sshKeyProtected", "sshEntryRoot", "sshIsWindows"}, "SSH settings not configured - run \"mutn init\" to configure") missingValueError = "SSH settings not configured - run \"mutn init\" to configure"
} else { } else {
sshUserConfig = backend.ReadConfig([]string{"sshUser", "sshIP", "sshPort", "sshKey", "sshKeyProtected"}, "0") missingValueError = "0"
} }
sshUserConfig = backend.ReadConfig([]string{"sshUser", "sshIP", "sshPort", "sshKey", "sshKeyProtected", "sshEntryRoot", "sshIsWindows"}, missingValueError)
var user, ip, port, keyFile, keyFileProtected, entryRoot string var user, ip, port, keyFile, keyFileProtected, entryRoot string
var isWindows bool var isWindows bool
@@ -299,7 +301,7 @@ func sftpSync(downloadList, uploadList []string, manualSync bool) {
var remoteFile *sftp.File var remoteFile *sftp.File
remoteFile, err = sftpClient.Create(remoteEntryFullPath) remoteFile, err = sftpClient.Create(remoteEntryFullPath)
if err != nil { if err != nil {
fmt.Println(backend.AnsiError+"Sync failed - Unable to create remote file:", err.Error()+backend.AnsiReset) fmt.Println(backend.AnsiError+"Sync failed - Unable to create remote file ("+remoteEntryFullPath+"):", err.Error()+backend.AnsiReset)
os.Exit(1) os.Exit(1)
} }