Closed woshilapin closed 11 months ago
Looks great! We only need one change, see https://github.com/orhun/rustypaste/issues/161#issuecomment-1769170790
Looks great! We only need one change, see #161 (comment)
@orhun Sure thing, here it is in a404dfc.
Attention: 20 lines
in your changes are missing coverage. Please review.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Files | Coverage Δ | |
---|---|---|
src/auth.rs | 90.90% <100.00%> (ø) |
|
src/middleware.rs | 72.00% <100.00%> (ø) |
|
src/config.rs | 82.85% <25.00%> (ø) |
|
src/server.rs | 82.87% <66.66%> (ø) |
|
src/main.rs | 0.00% <0.00%> (ø) |
... and 1 file with indirect coverage changes
:loudspeaker: Thoughts on this report? Let us know!.
LGTM, just one thing, can you configure tracing subscriber so that the default log level will be INFO
? Because now if you run rustypaste
then you get no output. We should somehow use INFO
as default to not break CLI backwards compatibility.
LGTM, just one thing, can you configure tracing subscriber so that the default log level will be
INFO
? Because now if you runrustypaste
then you get no output. We should somehow useINFO
as default to not break CLI backwards compatibility.
@orhun Good point. Done in b18610c thanks to https://docs.rs/tracing-subscriber/latest/tracing_subscriber/filter/struct.EnvFilter.html#method.from_default_env.
resolves #161
Description
Only changing
env_logger
intotracing-subscriber
. It should support the same configuration withRUST_LOG
(and more, if we convert fromlog
totracing
later).Motivation and Context
See #161.
How Has This Been Tested?
Logging are usually not tested. If any test would be needed, could you give some pointers?
Changelog Entry
-->
Types of Changes
Checklist: