chore: include logpath in case of logflush failure

ALX99 · May 19, 2024 · f124af9 · f124af9
1 parent b319180
commit f124af9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 10 deletions.
diff --git a/cmd/sail/main.go b/cmd/sail/main.go
@@ -42,19 +42,14 @@ func main() {
 
 	flush := util.SetupLogger(!isDev)
 	defer func() {
-		if err = flush(); err != nil {
-			fmt.Fprintln(os.Stderr, "failed to flush log: "+err.Error())
+		if logPath, err := flush(); err != nil {
+			fmt.Fprintf(os.Stderr, "Failed flushing logs to %q: %v\n", logPath, err)
 		}
 	}()
 	log.Info().Msg("Sail started")
 
 	util.SetupStyles()
 
-	// m, err := primary.New(state.NewState(), primary.Config{PWDFile: *pwdFile}, cfg)
-	// if err != nil {
-	// 	log.Fatal().Err(err).Send()
-	// }
-
 	cwd, err := os.Getwd()
 	if err != nil {
 		fmt.Fprintf(os.Stderr, "%v\n", err)

diff --git a/internal/util/logger.go b/internal/util/logger.go
@@ -15,19 +15,19 @@ import (
 )
 
 // SetupLogger sets up the global logger
-func SetupLogger(buffered bool) (flush func() error) {
+func SetupLogger(buffered bool) (flush func() (string, error)) {
 	fPath := path.Join(os.TempDir(), "sail.log")
 	f, err := os.Create(fPath)
 	if err != nil {
 		panic(err)
 	}
 
 	var w io.Writer = f
-	flush = func() error { return nil }
+	flush = func() (string, error) { return fPath, nil }
 
 	if buffered {
 		w = bufio.NewWriter(f)
-		flush = w.(*bufio.Writer).Flush
+		flush = func() (string, error) { return fPath, w.(*bufio.Writer).Flush() }
 	}
 
 	log.Logger = log.Output(zerolog.ConsoleWriter{