search

leodido / caddy-conditional-logging

Caddy log encoder for conditional logging
Apache License 2.0
6 stars 1 forks source link

Child encoder module is hardcoded to only 3 built-in ones #6

Open francislavoie opened 2 years ago

francislavoie commented 2 years ago

I told some users in https://github.com/caddyserver/transform-encoder/issues/18#issuecomment-1257299074 that they could use this plugin along with the transform encoder, but apparently this plugin doesn't allow it because of the switch here:

https://github.com/leodido/caddy-conditional-logging/blob/8534d30877aa34bdc083ef3cf466821fa3ade019/plugin.go#L171

This plugin should allow any encoder module to be a child.

(That said, there's now a skip_log directive built into Caddy since v2.6 so this plugin probably isn't really needed anymore)