Conversation
|
👋 wentzeld, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
|
IMHO most of these messages are way too verbose for low level errors. They read like high level log messages, by making assumptions about what is going on more broadly and by advising a node operator or a user what to do. This is an inversion of responsibilities - it is the caller's responsibility to determine how to handle low lever errors, and whether to wrap them up with additional details. Some of this information can also exist in documentation rather than as noise in the logs. |
Changes: Fee errors (ErrBumpFeeExceedsLimit, ErrConnectivity, ErrFeeLimitTooLow), multinode ErrNodeError, WriteTarget transmission errors, and corresponding test updates
Dependencies: None
See CRE Error Improvements