chore_: pass logger to backup controller

Author: jrainville

node/backup/controller.go

@@ -1,12 +1,14 @@
 package backup
 
 import (
-	"fmt"
+	"errors"
 	"os"
 	"path/filepath"
 	"sync"
 	"time"
 
+	"go.uber.org/zap"
+
 	"github.com/status-im/status-go/common"
 )
 
@@ -20,21 +22,23 @@ type BackupConfig struct {
 type Controller struct {
 	config BackupConfig
 	core   *core
+	logger *zap.Logger
 	quit   chan struct{}
 	mutex  sync.Mutex
 }
 
-func NewController(config BackupConfig) (*Controller, error) {
+func NewController(config BackupConfig, logger *zap.Logger) (*Controller, error) {
 	if len(config.PrivateKey) == 0 {
-		return nil, fmt.Errorf("private key must be provided")
+		return nil, errors.New("private key must be provided")
 	}
 	if config.FileNameGetter == nil {
-		return nil, fmt.Errorf("filename getter must be provided")
+		return nil, errors.New("filename getter must be provided")
 	}
 
 	return &Controller{
 		config: config,
 		core:   newCore(),
+		logger: logger,
 		quit:   make(chan struct{}),
 	}, nil
 }
@@ -60,8 +64,7 @@ func (c *Controller) Start() {
 			case <-ticker.C:
 				_, err := c.PerformBackup()
 				if err != nil {
-					// TODDO use a logger
-					fmt.Printf("Error performing backup: %v\n", err)
+					c.logger.Error("Error performing backup: %v\n", zap.Error(err))
 				}
 			case <-c.quit:
 				return

node/get_status_node.go

@@ -234,7 +234,7 @@ func (n *StatusNode) StartLocalBackup() error {
 		// TODO set to true to enable the local backup
 		BackupEnabled: false,
 		Interval:      time.Minute * 30,
-	})
+	}, n.logger.Named("LocalBackup"))
 	if err != nil {
 		return err
 	}