How about removing ZAP?

[jptosso]

Zap is a huge library and it requires a lot of dependencies, I'm not sure if it is worth it to keep using zap.

• Are the alternatives, like logrus, faster?
• How many indirect dependencies would we remove?
• How much would be the size loss of a Coraza package?
• Is structured logging really faster than "concatenating strings"?
• Are there similar, better alternatives?

[nacx]

Another option is to consider something like https://github.com/tetratelabs/telemetry/blob/master/logger.go which is just an interface. This has zero dependencies and users would be free to choose their implementation. If there are other interface driven libraries, then that would be great.

[jptosso]

Hey @nacx, thank you for your comment. ZAP was already removed from v3 and replaced with a similar logging interface that can be easily replaced. #337

[jcchavezs]

We moved away from zap in v3 using std logger. We can close this discussion now.

…

On Wed, 31 Aug 2022, 18:20 Ignasi Barrera, ***@***.***> wrote: Another option is to consider something like https://github.com/tetratelabs/telemetry/blob/master/logger.go which is just an interface. This has zero dependencies and users would be free to choose their implementation. If there are other interface driven libraries, then that would be great. — Reply to this email directly, view it on GitHub <#121 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAXOYAS6JP65RHLRQB3LRSLV36A3BANCNFSM5JQAIFGQ> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>