Merge pull request #9 from buty4649/improve-debug-logger

Add logger to iproute2 and update slog usage

cmd/root.go

@@ -63,18 +63,20 @@ var rootCmd = &cobra.Command{
 		if flags.Quiet {
 			logLevel = slog.LevelError
 		}
-		slog.SetDefault(slog.New(slogor.NewHandler(os.Stdout, slogor.Options{
+		logger := slog.New(slogor.NewHandler(os.Stdout, slogor.Options{
 			TimeFormat: "",
 			Level:      logLevel,
 			ShowSource: false,
-		})))
+		}))
+		slog.SetDefault(logger)
 
 		cfg, err = config.LoadConfig(flags.ConfigPath)
 		if err != nil {
 			return err
 		}
 
 		ip = iproute2.New(flags.IpCmdPath)
+		iproute2.SetLogger(logger)
 		return nil
 	},
 }

iproute2/iproute2.go

@@ -43,6 +43,8 @@ type Error struct {
 	Message    string
 }
 
+var logger *slog.Logger
+
 func New(path string) *Iproute2 {
 	return &Iproute2{
 		path:     path,
@@ -51,6 +53,10 @@ func New(path string) *Iproute2 {
 	}
 }
 
+func SetLogger(l *slog.Logger) {
+	logger = l
+}
+
 func (i *Iproute2) AddLink(name string, linkType string, options ...string) error {
 	args := []string{"link", "add", "name", name, "type", linkType}
 	args = append(args, options...)
@@ -189,7 +195,9 @@ func (i *Iproute2) executeWithStdout(args ...string) (string, error) {
 	}
 	cmdArgs = append(cmdArgs, args...)
 
-	slog.Debug(fmt.Sprintf("exec: %s %s", i.path, strings.Join(cmdArgs, " ")))
+	if logger != nil {
+		logger.Debug("exec command", "path", i.path, "args", strings.Join(cmdArgs, " "))
+	}
 
 	cmd := exec.Command(i.path, cmdArgs...)
 	var stdoutBuf, stderrBuf bytes.Buffer