X-Git-Url: https://ruderich.org/simon/gitweb/?p=nsscash%2Fnsscash.git;a=blobdiff_plain;f=main.go;h=5455dfe0b367cc8bf92f2ba20979273a97dce086;hp=db21997fee0cf71bdcfe5ed2836b7b092ea5b10b;hb=d946c291618d93e4ab4ad898c1ae317fbe2256e3;hpb=0f368ece4ec73bc058d5fb1459221ffd860d3bf3

diff --git a/main.go b/main.go
index db21997..5455dfe 100644
--- a/main.go
+++ b/main.go
@@ -24,6 +24,7 @@ import (
 	"io/ioutil"
 	"log"
 	"os"
+	"path/filepath"
 )
 
 func main() {
@@ -49,19 +50,7 @@ func main() {
 			break
 		}
 
-		cfg, err := LoadConfig(args[1])
-		if err != nil {
-			log.Fatal(err)
-		}
-		state, err := LoadState(cfg.StatePath)
-		if err != nil {
-			log.Fatal(err)
-		}
-		err = handleFiles(cfg, state)
-		if err != nil {
-			log.Fatal(err)
-		}
-		err = WriteStateIfChanged(cfg.StatePath, state)
+		err := mainFetch(args[1])
 		if err != nil {
 			log.Fatal(err)
 		}
@@ -72,51 +61,98 @@ func main() {
 			break
 		}
 
-		var t FileType
-		err := t.UnmarshalText([]byte(args[1]))
+		err := mainConvert(args[1], args[2], args[3])
 		if err != nil {
 			log.Fatal(err)
 		}
+		return
+	}
 
-		src, err := ioutil.ReadFile(args[2])
-		if err != nil {
-			log.Fatal(err)
-		}
-		var x bytes.Buffer
-		if t == FileTypePlain {
-			x.Write(src)
-		} else if t == FileTypePasswd {
+	flag.Usage()
+	os.Exit(1)
+}
+
+func mainFetch(cfgPath string) error {
+	cfg, err := LoadConfig(cfgPath)
+	if err != nil {
+		return err
+	}
+	state, err := LoadState(cfg.StatePath)
+	if err != nil {
+		return err
+	}
+	err = handleFiles(cfg, state)
+	if err != nil {
+		return err
+	}
+	// NOTE: Make sure to call WriteState() only if there were no
+	// errors (see WriteState() and README)
+	err = WriteState(cfg.StatePath, state)
+	if err != nil {
+		return err
+	}
+	return nil
+}
+
+func mainConvert(typ, srcPath, dstPath string) error {
+	var t FileType
+	err := t.UnmarshalText([]byte(typ))
+	if err != nil {
+		return err
+	}
+
+	src, err := ioutil.ReadFile(srcPath)
+	if err != nil {
+		return err
+	}
+	var x bytes.Buffer
+	if t == FileTypePlain {
+		x.Write(src)
+	} else if t == FileTypePasswd {
 		pws, err := ParsePasswds(bytes.NewReader(src))
 		if err != nil {
-			log.Fatal(err)
+			return err
 		}
 		err = SerializePasswds(&x, pws)
 		if err != nil {
-			log.Fatal(err)
+			return err
 		}
-		} else {
-			log.Fatalf("unsupported file type %v", t)
-		}
-
-		// We must create the file first or deployFile() will abort
-		f, err := os.Create(args[3])
+	} else if t == FileTypeGroup {
+		grs, err := ParseGroups(bytes.NewReader(src))
 		if err != nil {
-			log.Fatal(err)
+			return err
 		}
-		f.Close()
-
-		err = deployFile(&File{
-			Type: t,
-			Url:  args[2],
-			Path: args[3],
-			body: x.Bytes(),
-		})
+		err = SerializeGroups(&x, grs)
 		if err != nil {
-			log.Fatal(err)
+			return err
 		}
-		return
+	} else {
+		return fmt.Errorf("unsupported file type %v", t)
 	}
 
-	flag.Usage()
-	os.Exit(1)
+	dstDir := filepath.Dir(dstPath)
+	dstName := filepath.Base(dstPath)
+
+	// We must create the file first or deployFile() will abort; this is
+	// ugly because deployFile() already performs an atomic replacement
+	// but the simplest solution with the least duplicate code
+	f, err := ioutil.TempFile(dstDir, "tmp-"+dstName+"-")
+	if err != nil {
+		return err
+	}
+	tmpPath := f.Name()
+	defer os.Remove(tmpPath)
+	f.Close()
+
+	err = deployFile(&File{
+		Type: t,
+		Url:  srcPath,
+		Path: tmpPath,
+		body: x.Bytes(),
+	})
+	if err != nil {
+		return err
+	}
+
+	return os.Rename(tmpPath, dstPath)
 }