replace slogrus with log/slog

[uhthomas]

slogrus is not particularly efficient and is an unnecessary external dependency which can be avoided with log/slog.

[onedr0p]

The unifi webhook just migrated to uber/zap. Besides performance, the logo won my choice for a logging library.

kashalls/external-dns-unifi-webhook@6bc8b84

[uhthomas]

Zap admittedly does have good performance, but I can guarantee logging is the least of any performance concerns. My concern here is dependency and standardisation. Their reasoning for migrating is that logrus is in maintenance mode and is therefore even more reason to replace logrus, but zap is not necessarily the obvious choice given log/slog does everything which could be needed and doesn't require an addition dependency. I really would urge the use of log/slog - it's the standard and external packages should only every be considered if performance is a concern (which it isn't). I say this as a long time user (uber-go/zap#736) and enjoyer of zap. It's just not necessary anymore.

Just to touch on performance, there isn't a meaningful difference between zap and log/slog which makes the maintenance burden worth it. Same reason every Go project uses net/http instead of fasthttp or other examples.

[onedr0p]

but does slog have a cool gopher logo? 🤣

[uhthomas]

but does slog have a cool gopher logo? 🤣

I suppose it inherits the normal Go gopher as it's a standard library package.