Can't use some keys in WithField/WithFields about logrus

[sirupsen]

You are right—the current behaviour isn't exactly nice. I'm leaning towards printing a warning with logrus in the case where either of those keys exist—and still override it.

Alternatives would be to prefix the builtin ones with _, but that's messy and I think in most cases people won't override those. The one you'd be most tempted to override feels like Level—but even that is confusing.

What do you think of that?

[aybabtme]

Because they're always there, I'd make them fields on the Entry struct instead of values in the map.

[sirupsen]

That makes sense—what happens if they're set in the map though? Ignored? Still printed?

[aybabtme]

Still printed, they'd be fields like any other.

[sirupsen]

We discussed this in person and decided to leave it as is for now.