Automatic error logging
decoursin opened this issue · comments
What is the problem this feature would solve?
More convenient logging for errors.
What is the feature you are proposing to solve the problem?
ZIO has the below logError
function, which will automatically log if the this
ZIO element is an error.
/**
* Logs the cause of failure of this workflow.
*/
final def logError(implicit trace: Trace): ZIO[R, E, A] =
logError("")
Effect
also has a logError
function which works completely differently. That's fine, but it would also be nice to have a function like the one above which will just automatically log any error for me.
What alternatives have you considered?
.pipe(Effect.tapError(e => Effect.logError(`name='${e.name}' message='${e.message}'`)))
No response
I think Loggers should probably just be extended to have better support for Error
types so that.pipe(Effect.tapError(Effect.logError))
is sufficient.
I would think it not to be best to have the logError
function have two purposes. Right now logError
means logging at the error level. Or otherwise that should then be done for all of the logging functions like logDebug
, logInfo
, etc or for none of them. I don't think logError
should be special.
That's what I mean, yeah. I think all loggers, logDebug, etc. should have better support for logging an instance of an Error
type if that's what's provided to it. #2667
Ah, ok! I think that could be cool, yeah!
They already have good support for Cause
effect.pipe(Effect.tapErrorCause(Effect.logError))