Logging improvements

Author: leocomelli

main.go

@@ -2,10 +2,14 @@ package main
 
 import (
 	"flag"
-	"log"
+	"fmt"
+
+	"go.uber.org/zap"
 )
 
 var (
+	logger, _ = zap.NewProduction()
+
 	providers = map[string]SecretProvider{
 		"gcp": &GCPSecretManager{},
 	}
@@ -36,7 +40,15 @@ type Options struct {
 	Output   string
 }
 
+func (o *Options) String() string {
+	return fmt.Sprintf(
+		"provider: %s, project: %s, filter: %s, parser: %s, template: %s, output: %s",
+		o.Provider, o.Project, o.Filter, o.Parser, o.Template, o.Output,
+	)
+}
+
 func main() {
+
 	options := &Options{}
 
 	flag.StringVar(&options.Provider, "provider", "gcp", "name of the provider that manages the secrets")
@@ -56,6 +68,6 @@ func main() {
 
 	err := Run(options)
 	if err != nil {
-		log.Fatal(err)
+		logger.Fatal("error getting the secrets", zap.Error(err))
 	}
 }

parser.go

@@ -3,6 +3,8 @@ package main
 import (
 	"encoding/json"
 	"fmt"
+
+	"go.uber.org/zap"
 )
 
 // ContentParser defines secret content parser behaviors
@@ -34,7 +36,7 @@ type JSONContentParser struct {
 func (j *JSONContentParser) Parse(s *SecretData) []*SecretData {
 	m := map[string]interface{}{}
 	if err := json.Unmarshal([]byte(s.Data), &m); err != nil {
-		fmt.Printf("\nWARN: [%s] invalid json\n", s.Name)
+		logger.Warn("invalid json", zap.String("name", s.Name))
 	}
 
 	var secrets []*SecretData

runner.go

@@ -3,6 +3,8 @@ package main
 import (
 	"errors"
 	"os"
+
+	"go.uber.org/zap"
 )
 
 var (
@@ -32,6 +34,8 @@ func Run(options *Options) error {
 
 	}
 
+	logger.Info("using options", zap.Any("values", options))
+
 	if err := provider.Init(); err != nil {
 		return err
 	}
@@ -41,6 +45,8 @@ func Run(options *Options) error {
 		return err
 	}
 
+	logger.Info("secrets found", zap.Int("len", len(data)))
+
 	output := os.Stdin
 	if options.Output != "" {
 		output, err = os.Create(options.Output)