Logging improvements

[ahopkins]

This replaces the nice work that @Tronic did on #2852 to include debug and prod style logging, as well as a legacy formatter to allow for backwards compat logging for some period of time.

Debug logging

Prod logging (w/ access logs explicitly on)

---

As you can see the main difference is just in the usage of the timestamp. We skip it in Debug (except for access logs).

[codecov]

Codecov Report

Attention: Patch coverage is 74.10359% with 65 lines in your changes are missing coverage. Please review.

Project coverage is 87.626%. Comparing base (1a70930) to head (9e19653).
Report is 1 commits behind head on main.

❗ Current head 9e19653 differs from pull request most recent head 818a67a. Consider uploading reports for the commit 818a67a to get more accurate results

Files	Patch %	Lines
sanic/server/protocols/websocket_protocol.py	28.571%	28 Missing and 2 partials ⚠️
sanic/logging/formatter.py	88.095%	6 Missing and 4 partials ⚠️
sanic/server/websockets/impl.py	33.333%	10 Missing ⚠️
sanic/server/protocols/http_protocol.py	25.000%	9 Missing ⚠️
sanic/logging/setup.py	84.210%	2 Missing and 1 partial ⚠️
sanic/worker/manager.py	85.714%	1 Missing and 1 partial ⚠️
sanic/http/http1.py	83.333%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@              Coverage Diff              @@
##              main     #2931       +/-   ##
=============================================
- Coverage   87.985%   87.626%   -0.360%     
=============================================
  Files           94       101        +7     
  Lines         7408      7597      +189     
  Branches      1307      1328       +21     
=============================================
+ Hits          6518      6657      +139     
- Misses         599       644       +45     
- Partials       291       296        +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[prryplatypus]

What's the reason for removing the timestamp? I personally find it useful while developing - helps spotting slow bits of code and correlating events between systems/apps (for example if you have a webserver and a background worker).

[ahopkins]

@prryplatypus Muddies up the logs more than necessary if not needed. If it is needed, it is not so difficult to get the timestamps:

from sanic.log import LOGGING_CONFIG_DEFAULTS

LOGGING_CONFIG_DEFAULTS['formatters']['generic']['class'] = 'sanic.logging.formatter.ProdFormatter'

Perhaps we could make it all even simpler with config values on the Sanic app. Something like:

app.config.LOG_FORMATTER = "sanic.logging.formatter.ProdFormatter"

FWIW, uvicorn doesn't even give you timestamps on access logs.

[ahopkins]

FWIW, even with the time in dev (no date, just time) we still pick up a lot of real estate and cleanliness.

[prryplatypus]

I'd be happy with just time 👀