X-Git-Url: https://git.r.bdr.sh/rbdr/mobius/blobdiff_plain/8796b4494547be6825004d82bf40db71524a6400..46acf0721f9bcd5304d0a0fbc10ebe544786d157:/cmd/mobius-hotline-server/main.go

diff --git a/cmd/mobius-hotline-server/main.go b/cmd/mobius-hotline-server/main.go
index cf3c5a9..8407449 100644
--- a/cmd/mobius-hotline-server/main.go
+++ b/cmd/mobius-hotline-server/main.go
@@ -14,6 +14,7 @@ import (
 	"math/rand"
 	"net/http"
 	"os"
+	"path/filepath"
 	"runtime"
 	"time"
 )
@@ -28,7 +29,22 @@ const (
 func main() {
 	rand.Seed(time.Now().UnixNano())
 
-	ctx, cancelRoot := context.WithCancel(context.Background())
+	ctx, cancel := context.WithCancel(context.Background())
+
+	// TODO: implement graceful shutdown by closing context
+	// c := make(chan os.Signal, 1)
+	// signal.Notify(c, os.Interrupt)
+	// defer func() {
+	// 	signal.Stop(c)
+	// 	cancel()
+	// }()
+	// go func() {
+	// 	select {
+	// 	case <-c:
+	// 		cancel()
+	// 	case <-ctx.Done():
+	// 	}
+	// }()
 
 	basePort := flag.Int("bind", defaultPort, "Bind address and port")
 	statsPort := flag.String("stats-port", "", "Enable stats HTTP endpoint on address and port")
@@ -56,24 +72,26 @@ func main() {
 	logger := l.Sugar()
 
 	if *init {
-		if _, err := os.Stat(*configDir + "/config.yaml"); !os.IsNotExist(err) {
-			logger.Fatalw("Init failed.  Existing config directory found: " + *configDir)
-		}
+		if _, err := os.Stat(filepath.Join(*configDir, "/config.yaml")); os.IsNotExist(err) {
+			if err := os.MkdirAll(*configDir, 0750); err != nil {
+				logger.Fatal(err)
+			}
 
-		if err := os.MkdirAll(*configDir, 0750); err != nil {
-			logger.Fatal(err)
-		}
+			if err := copyDir("mobius/config", *configDir); err != nil {
+				logger.Fatal(err)
+			}
+			logger.Infow("Config dir initialized at " + *configDir)
 
-		if err := copyDir("mobius/config", *configDir); err != nil {
-			logger.Fatal(err)
+		} else {
+			logger.Infow("Existing config dir found.  Skipping initialization.")
 		}
 	}
 
 	if _, err := os.Stat(*configDir); os.IsNotExist(err) {
-		logger.Fatalw("Configuration directory not found", "path", configDir)
+		logger.Fatalw("Configuration directory not found.  Correct the path or re-run with -init to generate initial config.", "path", configDir)
 	}
 
-	srv, err := hotline.NewServer(*configDir, "", *basePort, logger, &hotline.OSFileStore{})
+	srv, err := hotline.NewServer(*configDir, *basePort, logger, &hotline.OSFileStore{})
 	if err != nil {
 		logger.Fatal(err)
 	}
@@ -92,7 +110,7 @@ func main() {
 	}
 
 	// Serve Hotline requests until program exit
-	logger.Fatal(srv.ListenAndServe(ctx, cancelRoot))
+	logger.Fatal(srv.ListenAndServe(ctx, cancel))
 }
 
 type statHandler struct {
@@ -100,7 +118,7 @@ type statHandler struct {
 }
 
 func (sh *statHandler) RenderStats(w http.ResponseWriter, _ *http.Request) {
-	u, err := json.Marshal(sh.hlServer.Stats)
+	u, err := json.Marshal(sh.hlServer.CurrentStats())
 	if err != nil {
 		panic(err)
 	}
@@ -130,7 +148,7 @@ var zapLogLevel = map[string]zapcore.Level{
 func defaultConfigPath() (cfgPath string) {
 	switch runtime.GOOS {
 	case "windows":
-		cfgPath = "config"
+		cfgPath = "config/"
 	case "darwin":
 		if _, err := os.Stat("/usr/local/var/mobius/config/"); err == nil {
 			cfgPath = "/usr/local/var/mobius/config/"
@@ -154,17 +172,17 @@ func copyDir(src, dst string) error {
 	}
 	for _, dirEntry := range entries {
 		if dirEntry.IsDir() {
-			if err := os.MkdirAll(dst+"/"+dirEntry.Name(), 0777); err != nil {
+			if err := os.MkdirAll(filepath.Join(dst, dirEntry.Name()), 0777); err != nil {
 				panic(err)
 			}
-			subdirEntries, _ := cfgTemplate.ReadDir(src + "/" + dirEntry.Name())
+			subdirEntries, _ := cfgTemplate.ReadDir(filepath.Join(src, dirEntry.Name()))
 			for _, subDirEntry := range subdirEntries {
-				f, err := os.Create(dst + "/" + dirEntry.Name() + "/" + subDirEntry.Name())
+				f, err := os.Create(filepath.Join(dst, dirEntry.Name(), subDirEntry.Name()))
 				if err != nil {
 					return err
 				}
 
-				srcFile, err := cfgTemplate.Open(src + "/" + dirEntry.Name() + "/" + subDirEntry.Name())
+				srcFile, err := cfgTemplate.Open(filepath.Join(src, dirEntry.Name(), subDirEntry.Name()))
 				if err != nil {
 					return err
 				}
@@ -175,12 +193,12 @@ func copyDir(src, dst string) error {
 				f.Close()
 			}
 		} else {
-			f, err := os.Create(dst + "/" + dirEntry.Name())
+			f, err := os.Create(filepath.Join(dst, dirEntry.Name()))
 			if err != nil {
 				return err
 			}
 
-			srcFile, err := cfgTemplate.Open(src + "/" + dirEntry.Name())
+			srcFile, err := cfgTemplate.Open(filepath.Join(src, dirEntry.Name()))
 			if err != nil {
 				return err
 			}