[proxy] Produce access logs only for non-200 requests

[csweichel]

Thereby cleaning up the proxy logs and making them more useful.

[csweichel]

I'm not convinced that Caddy can do this out of the box. I found nothing in the docs our sources that would indicate conditional logging.

[leodido]

Probably there's a "tricky" way to do it with Caddy (need to double-check on its version, tho).

Setting the level version of the log config to warn would avoid 200s being logged.

https://caddyserver.com/docs/caddyfile/directives/log

---

I'm defining this "tricky" because it's more a side effect than an ability to effectively select what to log and what to do not log.

[aledbf]

I think we should not use the default logger and instead write a small plugin like this one and just set the fields we need.
Basically, trim this file and define a new directive, like gitpod.log

[leodido]

think we should not use the default logger and instead write a small plugin like this one and just set the fields we need.
Basically, trim this file and define a new directive, like gitpod.log

This ^^^ would be the correct way to go. I agree.

Just to clarify: I defined my attempt as a trick for this exact reason :)