VolantMQ / volantmq

High-Performance MQTT Server
Apache License 2.0
981 stars 169 forks source link

Allow VolantMQ to operate with Zap v1.16 #181

Closed argylelabcoat closed 2 years ago

argylelabcoat commented 3 years ago

I was using VolantMQ in a project where I have vendored dependencies and as a result, my Zap 1.16.0 overrode the VolantMQ Zap of 1.12

This causes an error to be thrown when creating the logger:

    log, err := logCfg.Build()

Due to the null TimeEncoder.

Since the intent seemed to be to have no time prefix on the logs, I made a change to allow for that while giving it a non-nil pointer.