changed logger to sirupsen/logrus

cmd/gosdn/main.go

 package main
 
 import (
-	"code.fbi.h-da.de/cocsn/gosdn/log"
 	"code.fbi.h-da.de/cocsn/gosdn/nucleus"
 	"flag"
+	log "github.com/sirupsen/logrus"
 )
 
 func main() {
@@ -16,7 +16,7 @@ func main() {
 	flag.Parse()
 	cliSocket := *cliListenAddr + ":" + *cliListenPort
 
-	log.Loglevel(log.DEBUG)
+	log.SetLevel(log.DebugLevel)
 
 	// Setup a channel to communicate if goSDN should shutdown.
 	IsRunningChannel := make(chan bool)

database/client.go

 package database
 
 import (
-	"code.fbi.h-da.de/cocsn/gosdn/log"
+	log "github.com/sirupsen/logrus"
 	"errors"
 	"github.com/neo4j/neo4j-go-driver/neo4j"
 )

go.mod

@@ -13,6 +13,7 @@ require (
 	github.com/neo4j/neo4j-go-driver v1.8.3
 	github.com/onsi/ginkgo v1.13.0 // indirect
 	github.com/rivo/tview v0.0.0-20201018122409-d551c850a743
+	github.com/sirupsen/logrus v1.4.2
 	golang.org/x/net v0.0.0-20200904194848-62affa334b73 // indirect
 	google.golang.org/genproto v0.0.0-20200519141106-08726f379972 // indirect
 	google.golang.org/grpc v1.29.1

go.sum

@@ -228,6 +228,7 @@ github.com/sclevine/agouti v3.0.0+incompatible/go.mod h1:b4WX9W9L1sfQKXeJf1mUTLZ
 github.com/sergi/go-diff v1.0.0/go.mod h1:0CfEIISq7TuYL3j771MWULgwwjU+GofnZX9QAmXWZgo=
 github.com/sirupsen/logrus v1.4.0/go.mod h1:LxeOpSwHxABJmUn/MG1IvRgCAasNZTLOkJPxbbu5VWo=
 github.com/sirupsen/logrus v1.4.1/go.mod h1:ni0Sbl8bgC9z8RoU9G6nDWqqs/fq4eDPysMBDgk/93Q=
+github.com/sirupsen/logrus v1.4.2 h1:SPIRibHv4MatM3XXNO2BJeFLZwZ2LvZgfQ5+UNI2im4=
 github.com/sirupsen/logrus v1.4.2/go.mod h1:tLMulIdttU9McNUspp0xgXVQah82FyeX6MwdIuYE2rE=
 github.com/spf13/cobra v0.0.3/go.mod h1:1l0Ry5zgKvJasoi3XT1TypsSe7PqH0Sj9dhYf7v3XqQ=
 github.com/spf13/pflag v1.0.3 h1:zPAT6CGy6wXeQ7NtTnaTerfKOsV6V6F8agHXFiazDkg=

log/logger.go

-package log
-
-import (
-	"fmt"
-	"io"
-	"log/syslog"
-	"os"
-	"reflect"
-	"runtime"
-	"strconv"
-	"strings"
-	"sync"
-	"time"
-)
-
-var logger *Logger
-var once sync.Once
-
-// Logger is a wrapper for log.Logger and provides
-// methods to enable and disable logging.
-type Logger struct {
-	DefaultWriter  io.Writer
-	LoglevelWriter map[Level]io.Writer
-	toSyslog       map[Level]bool
-	Loglevel       Level
-	lock           sync.Mutex
-	builder        strings.Builder
-}
-
-func (l *Logger) buildMessage(level Level, syslog bool, args ...interface{}) {
-	if !syslog {
-		l.builder.WriteString(time.Now().Format(time.RFC3339))
-	}
-	l.builder.WriteRune('\t')
-	l.builder.WriteString(prefix(level))
-	l.builder.WriteRune('\t')
-	function, line := callers()
-	functionSplitted := strings.SplitAfter(function, "/")
-	function = functionSplitted[len(functionSplitted)-1]
-	l.builder.WriteString(function)
-	l.builder.WriteRune(':')
-	l.builder.WriteString(strconv.Itoa(line))
-	l.builder.WriteRune('\t')
-	l.builder.WriteString(fmt.Sprint(args...))
-	l.builder.WriteRune('\n')
-}
-
-func get() *Logger {
-	once.Do(func() {
-		logger = &Logger{
-			DefaultWriter:  os.Stderr,
-			LoglevelWriter: make(map[Level]io.Writer),
-			toSyslog:       make(map[Level]bool),
-			Loglevel:       INFO,
-			lock:           sync.Mutex{},
-		}
-	})
-	return logger
-}
-
-// Loglevel sets the verbosity of the logger
-// Defaults to INFO
-func Loglevel(level Level) {
-	l := get()
-	l.lock.Lock()
-	defer l.lock.Unlock()
-	l.Loglevel = level
-}
-
-// Output defines the output of the logger
-// Defaults to os.Stderr
-func Output(out io.Writer) {
-	l := get()
-	l.lock.Lock()
-	defer l.lock.Unlock()
-	l.DefaultWriter = out
-}
-
-// LoglevelOutput defines a special output
-// for a certain log level
-func LoglevelOutput(level Level, out io.Writer) {
-	l := get()
-	l.lock.Lock()
-	defer l.lock.Unlock()
-	l.LoglevelWriter[level] = out
-	if reflect.TypeOf(out) == reflect.TypeOf(&syslog.Writer{}) {
-		l.toSyslog[level] = true
-	}
-}
-
-// Debug passes the DEBUG flag and a
-// message to the logger
-func Debug(args ...interface{}) {
-	log(DEBUG, args...)
-}
-
-// Info passes the INFO flag and a
-// message to the logger
-func Info(args ...interface{}) {
-	log(INFO, args...)
-}
-
-// Warn passes the WARNING flag and a
-// message to the logger
-func Warn(args ...interface{}) {
-	log(WARNING, args...)
-}
-
-// Error passes the ERROR flag and a
-// message to the logger
-func Error(args ...interface{}) {
-	log(ERROR, args...)
-}
-
-// Fatal passes the FATAL flag and a
-// message to the logger and calls
-// os.Exit(1)
-func Fatal(args ...interface{}) {
-	log(FATAL, args...)
-	os.Exit(1)
-}
-
-// Debugf passes the DEBUG flag,
-// a formatted string and a
-// message to the logger
-func Debugf(format string, args ...interface{}) {
-	logf(DEBUG, format, args...)
-}
-
-// Infof passes the INFO flag,
-// a formatted string and and a
-// message to the logger
-func Infof(format string, args ...interface{}) {
-	logf(INFO, format, args...)
-}
-
-// Warnf passes the WARNING flag,
-// a formatted string and and a
-// message to the logger
-func Warnf(format string, args ...interface{}) {
-	logf(WARNING, format, args...)
-}
-
-// Errorf passes the ERROR flag,
-// a formatted string and and a
-// message to the logger
-func Errorf(format string, args ...interface{}) {
-	logf(ERROR, format, args...)
-}
-
-// Fatalf passes the FATAL flag,
-// a formatted string and and a
-// message to the logger and calls
-// os.Exit(1)
-func Fatalf(format string, args ...interface{}) {
-	logf(FATAL, format, args...)
-	os.Exit(1)
-}
-
-func logf(level Level, format string, args ...interface{}) {
-	log(level, fmt.Sprintf(format, args...))
-}
-
-func log(level Level, args ...interface{}) {
-	defer func() {
-		if r := recover(); r != nil {
-			fmt.Println("Recovered in f", r)
-		}
-	}()
-	l := get()
-	l.lock.Lock()
-	defer l.lock.Unlock()
-	defer l.builder.Reset()
-	if level <= l.Loglevel {
-		l.buildMessage(level, l.toSyslog[level], args...)
-		msg := []byte(l.builder.String())
-		writer, ok := l.LoglevelWriter[level]
-		var err error
-		if !ok {
-			_, err = l.DefaultWriter.Write(msg)
-		} else {
-			_, err = writer.Write(msg)
-		}
-		if err != nil {
-			panic(err)
-		}
-	}
-}
-
-func callers() (string, int) {
-	pc := make([]uintptr, 15)
-	n := runtime.Callers(5, pc)
-	frames := runtime.CallersFrames(pc[:n])
-	frame, _ := frames.Next()
-	return frame.Function, frame.Line
-}
-
-func prefix(level Level) string {
-	switch level {
-	case FATAL:
-		return "FATAL"
-	case ERROR:
-		return "ERROR"
-	case WARNING:
-		return "WARNING"
-	case INFO:
-		return "INFO"
-	case DEBUG:
-		return "DEBUG"
-	}
-	return ""
-}

log/loglevel.go

-package log
-
-// Level is an 8 bit integer representing a
-// log level for the logger
-type Level uint8
-
-// Constants for more verbose integer
-// values
-const (
-	FATAL Level = iota
-	ERROR
-	WARNING
-	INFO
-	DEBUG
-)

nucleus/cli-handling.go

@@ -14,8 +14,8 @@ import (
 	"sync"
 
 	pb "code.fbi.h-da.de/cocsn/gosdn/api/proto"
-	"code.fbi.h-da.de/cocsn/gosdn/log"
 	"code.fbi.h-da.de/cocsn/gosdn/sbi/restconf/client/ciena"
+	log "github.com/sirupsen/logrus"
 	"google.golang.org/grpc"
 	"google.golang.org/grpc/health"
 	healthpb "google.golang.org/grpc/health/grpc_health_v1"
@@ -120,7 +120,7 @@ func getCLIGoing(core *Core) {
 
 	//TODO: move?
 	wrt := io.MultiWriter(os.Stdout, &logBuffer)
-	log.Output(wrt)
+	log.SetOutput(wrt)
 
 	healthpb.RegisterHealthServer(cliControlServer, healthCheck)
 	pb.RegisterGrpcCliServer(cliControlServer, srv)

nucleus/controller.go

@@ -2,7 +2,7 @@ package nucleus
 
 import (
 	"code.fbi.h-da.de/cocsn/gosdn/database"
-	"code.fbi.h-da.de/cocsn/gosdn/log"
+	log "github.com/sirupsen/logrus"
 	"code.fbi.h-da.de/cocsn/gosdn/nucleus/interfaces"
 	"code.fbi.h-da.de/cocsn/gosdn/sbi/restconf/client/ciena"
 	"github.com/BurntSushi/toml"

nucleus/nucleus-core.go

@@ -2,7 +2,7 @@ package nucleus
 
 import (
 	"code.fbi.h-da.de/cocsn/gosdn/database"
-	"code.fbi.h-da.de/cocsn/gosdn/log"
+	log "github.com/sirupsen/logrus"
 	"code.fbi.h-da.de/cocsn/gosdn/nucleus/interfaces"
 	"time"
 )

sbi/restconf/client/ciena/client.go

@@ -3,7 +3,7 @@ package ciena
 import (
 	"bytes"
 	"code.fbi.h-da.de/cocsn/gosdn/database"
-	"code.fbi.h-da.de/cocsn/gosdn/log"
+	log "github.com/sirupsen/logrus"
 	"code.fbi.h-da.de/cocsn/gosdn/nucleus/interfaces"
 	"code.fbi.h-da.de/cocsn/gosdn/sbi/restconf/util"
 	apiclient "code.fbi.h-da.de/cocsn/swagger/apis/mcp/client"