js-libp2p
js-libp2p copied to clipboard
chore: use %e token anywhere we are logging errors with @libp2p/logger
Title
Description
Notes & open questions
Change checklist
- [x] I have performed a self-review of my own code
- [ ] I have made corresponding changes to the documentation if necessary (this includes comments as well)
- [ ] I have added tests that prove my fix is effective or that my feature works
@SuchitraSwain thanks for the PR - for next steps. Please address the review comments left here
@achingbrain @SgtPooki could you please approve this PR
@SuchitraSwain there is still an unrelated formatting change here, see the unresolved comment.
There are also still many instances of passing errors to a logger that have not had the %e token added to them left in the codebase so this PR does not completely address the issue yet.
Checking out again, will address all the error instance
Thanks for opening this PR. I'm closing it due to lack of activity in the last two months and because #3261 has been opened which adds the %e token to the necessary locations as requested here.