optimze package and remove logging subpackage

go.mod

@@ -1,7 +1,10 @@
-module github.com/tecamino/tecamino-logger
+module gitea.tecamino.local/paadi/tecamino-logger
 
 go 1.21.0
 
-require go.uber.org/zap v1.27.0
+require (
+	go.uber.org/zap v1.27.0
+	gopkg.in/natefinch/lumberjack.v2 v2.2.1
+)
 
 require go.uber.org/multierr v1.10.0 // indirect

go.sum

@@ -10,5 +10,7 @@ go.uber.org/multierr v1.10.0 h1:S0h4aNzvfcFsC3dRF1jLoaov7oRaKqRGC/pUEJ2yvPQ=
 go.uber.org/multierr v1.10.0/go.mod h1:20+QtiLqy0Nd6FdQB9TLXag12DsQkrbs3htMFfDN80Y=
 go.uber.org/zap v1.27.0 h1:aJMhYGrd5QSmlpLMr2MftRKl7t8J8PTZPA732ud/XR8=
 go.uber.org/zap v1.27.0/go.mod h1:GB2qFLM7cTU87MWRP2mPIjqfIDnGu+VIO4V/SdhGo2E=
+gopkg.in/natefinch/lumberjack.v2 v2.2.1 h1:bBRl1b0OH9s/DuPhuXpNl+VtCaJXFZ5/uEFST95x9zc=
+gopkg.in/natefinch/lumberjack.v2 v2.2.1/go.mod h1:YD8tP3GAjkrDg1eZH7EGmyESg/lsYskCTPBJVb9jqSc=
 gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA=
 gopkg.in/yaml.v3 v3.0.1/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=

logger/logger.go

@@ -1,62 +0,0 @@
-package logger
-
-import (
-	"fmt"
-	"os"
-	"time"
-
-	"go.uber.org/zap"
-	"go.uber.org/zap/zapcore"
-)
-
-type Level zapcore.Level
-
-const (
-	Info    Level = 0
-	Warning Level = 1
-	Error   Level = 2
-	Debug   Level = -1
-)
-
-type Logger struct {
-	log   *zap.Logger
-	debug *zap.Logger
-}
-
-func NewLogger(name string) (*Logger, error) {
-
-	if name == "" {
-		name = "NewLogger"
-	}
-	file, err := os.OpenFile(fmt.Sprintf("%s.log", name), os.O_APPEND|os.O_CREATE|os.O_WRONLY, 0644)
-	if err != nil {
-		return nil, err
-	}
-
-	encoderConfig := zap.NewProductionEncoderConfig()
-	encoderConfig.TimeKey = "timestamp"
-	encoderConfig.EncodeTime = func(t time.Time, enc zapcore.PrimitiveArrayEncoder) {
-		enc.AppendString(t.Format("2006-01-02T15:04:05.000"))
-	}
-	encoder := zapcore.NewJSONEncoder(encoderConfig)
-
-	// Build the logger
-	return &Logger{
-			log:   zap.New(zapcore.NewCore(encoder, zapcore.AddSync(file), zapcore.InfoLevel)),
-			debug: zap.New(zapcore.NewCore(encoder, zapcore.AddSync(file), zapcore.DebugLevel)),
-		},
-		nil
-}
-
-func (l *Logger) Log(logLevel Level, caller, msg string) {
-	switch logLevel {
-	case Info:
-		l.log.Info(msg, zap.String("caller", caller))
-	case Warning:
-		l.log.Warn(msg, zap.String("caller", caller))
-	case Error:
-		l.log.Error(msg, zap.String("caller", caller))
-	case Debug:
-		l.debug.Debug(msg, zap.String("caller", caller))
-	}
-}

logger_test.go

@@ -0,0 +1,40 @@
+package tecaminologger
+
+import (
+	"errors"
+	"os"
+	"testing"
+)
+
+func TestLogger(t *testing.T) {
+	t.Log("start logger test")
+	logFile := "Test.log"
+	log, err := NewLogger(logFile, nil)
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	log.Info("line 10", "hallo")
+	log.Warning("line 11", "vello")
+	log.Error("line 12", "hie")
+	err = errors.New("test error")
+	log.Error("line 14", err)
+	log.Debug("line 15", "isch")
+
+	t.Log("read created log file")
+	f, err := os.ReadFile(logFile)
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	t.Log("read created log file")
+	t.Log(string(f))
+
+	t.Log("test if log file name empty")
+	log, err = NewLogger("", nil)
+	if err != nil {
+		t.Fatal(err)
+	}
+	log.Info("test", "this is a text")
+
+}

main.go

@@ -1,14 +0,0 @@
-package main
-
-import "github.com/tecamino/tecamino-logger/logger"
-
-func main() {
-	log, err := logger.NewLogger("Test")
-	if err != nil {
-		panic(err)
-	}
-	log.Log(logger.Info, "line 10", "hallo")
-	log.Log(logger.Warning, "line 11", "vello")
-	log.Log(logger.Error, "line 12", "hie")
-	log.Log(logger.Debug, "line 13", "isch")
-}