Brad Moylan

This change is [](https://reviewable.io/reviews/palantir/witchcraft-go-server/694)

This change is [](https://reviewable.io/reviews/palantir/witchcraft-go-server/579)

Benchmarks show zerolog about 30% faster than zap with a smaller dependency tree. --- This change is [](https://reviewable.io/reviews/palantir/witchcraft-go-server/315)

Previously, all user-provided healthchecks must be provided in a single call to WithHealth. This has forced applications to build their own healthcheck accumulators. I have written this code in multiple...

Time-based log rotation

4

Some log archival tools only operate on rotated/compressed logfiles. For services which do not log very often, it can take weeks/months to reach the 1GB limit to trigger a rotation....

Today, if the `metric-emit-frequency` is set to `0` we use the default of 60s. This is different than `trace-sample-rate`, which disables logging of

This could give clients more helpful errors if we know an endpoint will fail cc @gdearment

Configuring the server on port 0 tells the kernel to assign a random port and return the address. We could handle this more nicely: - easy: pass instantiated listener to...

Custom yaml unmarshaling (e.g. defaults)

3

witchcraft calls yaml.Unmarshal for you, but this makes custom handling like loading defaults difficult.

This change is [](https://reviewable.io/reviews/palantir/pkg/222)