Merge pull request #1 from rjbasitali/logfmt

Logfmt

fields.go

@@ -0,0 +1,12 @@
+package log
+
+func (l myLogger) WithFields(fields Fields) Logger {
+	if l.data != nil && len(l.data) > 0 {
+		for k, v := range fields {
+			l.data[k] = v
+		}
+	} else {
+		l.data = fields
+	}
+	return l
+}

logger.go

@@ -29,4 +29,6 @@ type Logger interface {
 	End(a ...interface{})
 
 	Level(uint8) Logger
+
+	WithFields(fields Fields) Logger
 }

logger_begin.go

@@ -6,6 +6,6 @@ func (l myLogger) Begin(s ...interface{}) Logger {
 	if hasLevel(l.level, logFlag) {
 		l.log(logFlag, append([]interface{}{"BEGIN"}, s...)...)
 	}
-	logger := myLogger{Writer: l.Writer, prefix: l.prefix, begin: time.Now(), level: l.level}
+	logger := myLogger{Writer: l.Writer, ErrWriter: l.ErrWriter, prefix: l.prefix, begin: time.Now(), level: l.level, data: l.data}
 	return logger
 }

logger_level.go

@@ -20,7 +20,7 @@ const (
 // Anything above 6 as level will be considered Level 6.
 // Pass 0 to output no logs.
 func (l myLogger) Level(level uint8) Logger {
-	logger := myLogger{Writer: l.Writer, prefix: l.prefix, begin: l.begin}
+	logger := myLogger{Writer: l.Writer, ErrWriter: l.ErrWriter, prefix: l.prefix, begin: l.begin, data: l.data}
 	switch level {
 	case 1:
 		logger.level = alertFlag

logger_prefix.go

@@ -11,6 +11,6 @@ func (l myLogger) Prefix(p ...string) Logger {
 	for _, prefix := range p {
 		buffer.WriteString(fmt.Sprintf("%s: ", prefix))
 	}
-	logger := myLogger{Writer: l.Writer, prefix: buffer.String()}
+	logger := myLogger{Writer: l.Writer, ErrWriter: l.ErrWriter, prefix: buffer.String(), data: l.data}
 	return logger
 }

my_logger.go

@@ -13,33 +13,62 @@ var (
 	DefaultTimeFormat = time.RFC3339
 )
 
+type Fields map[string]interface{}
+
 type myLogger struct {
 	Writer    io.Writer
+	ErrWriter io.Writer
 	prefix    string
 	begin     time.Time
 	level     uint8
+	data      Fields
 }
 
 func logPrefix(level uint8) string {
 	return fmt.Sprintf("time=%q level=%s function=%s", time.Now().Format(DefaultTimeFormat), unmarshalLevel(level), funcName())
 }
 
-func (l myLogger) log(level uint8, s ...interface{}) {
+func (fields Fields) String() string {
-	if l.Writer == nil {
+	if len(fields) == 0 {
-		return
+		return ""
 	}
-	f := fmt.Sprintf("%s%s msg=%q", logPrefix(level), l.prefix, fmt.Sprint(s...))
+	var buffer strings.Builder
-	fmt.Fprintln(l.Writer, f)
+	for k, v := range fields {
+		switch v := v.(type) {
+		case string:
+			buffer.WriteString(fmt.Sprintf(" %s=%q", k, v))
+		default:
+			buffer.WriteString(fmt.Sprintf(" %s=%v", k, v))
+		}
+	}
+	return buffer.String()
 }
 
-func (l myLogger) logf(level uint8, format string, s ...interface{}) {
+func (l myLogger) log(flag uint8, s ...interface{}) {
-	if l.Writer == nil {
+	var w io.Writer
-		return
+	{
+		if flag == errorFlag {
+			w = l.ErrWriter
+		} else {
+			w = l.Writer
+		}
+	}
+	f := fmt.Sprintf("%s%s msg=%q%s", logPrefix(flag), l.prefix, fmt.Sprint(s...), l.data)
+	fmt.Fprintln(w, f)
 }
-	format = fmt.Sprintf("%%s %%s msg=\"%s\"\n", format)
-	s = append([]interface{}{logPrefix(level), l.prefix}, s...)
 
-	fmt.Fprintf(l.Writer, format, s...)
+func (l myLogger) logf(flag uint8, format string, s ...interface{}) {
+	var w io.Writer
+	{
+		if flag == errorFlag {
+			w = l.ErrWriter
+		} else {
+			w = l.Writer
+		}
+	}
+	format = fmt.Sprintf("%%s%%s msg=\"%s\"%s\n", format, l.data)
+	s = append([]interface{}{logPrefix(flag), l.prefix}, s...)
+	fmt.Fprintf(w, format, s...)
 }
 
 func funcName() string {
@@ -59,12 +88,16 @@ func funcName() string {
 	return ""
 }
 
-func NewLogger(w io.Writer) Logger {
+func NewLogger(w, errw io.Writer) Logger {
 	if w == nil {
 		w = os.Stdout
 	}
+	if errw == nil {
+		errw = os.Stderr
+	}
 	return myLogger{
 		Writer:    w,
+		ErrWriter: errw,
 		begin:     time.Now(),
 		level:     traceFlag,
 	}