-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Closed
Labels
good first issueDenotes an issue ready for a new contributor, according to the "help wanted" guidelines.Denotes an issue ready for a new contributor, according to the "help wanted" guidelines.
Description
we have the package internal/log to execute hooks and give more debugging information when we log. specifically, it's useful because it tracks a unique ID for each request, and it makes tracking the lifecycle of logs easier
however, to do so, it requires us to pass the context everywhere. We haven't wired it all over the place, so many places use logrus instead of log.
with #4650 we're passing the context to more places, and as such we should add a follow up to use the log package in more places.
without looking too hard at the code, i'd say we should use the log package in every function that currently takes a context
saschagrunert
Metadata
Metadata
Assignees
Labels
good first issueDenotes an issue ready for a new contributor, according to the "help wanted" guidelines.Denotes an issue ready for a new contributor, according to the "help wanted" guidelines.