a8m / a8m/kinesis-producer

Logrus Entry vs Logger

Offen
#23 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Go
Sterne
150
Forks
47
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

To support logging already defined fields using logrus, is there any concern with switching to `Entry` rather than `Logger`?

From [logrus docs](https://github.com/sirupsen/logrus/blob/master/entry.go#L47):

```
These objects can be reused and passed around as much as you wish to avoid field duplication.
```

Add [README](https://github.com/sirupsen/logrus#default-fields)

```
Often it's helpful to have fields always attached to log statements in an application or parts of one. For example, you may want to always log the request_id and user_ip in the context of a request. Instead of writing log.WithFields(log.Fields{"request_id": request_id, "user_ip": user_ip}) on every line, you can create a logrus.Entry to pass around instead:
```

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.