Thanks to visit codestin.com
Credit goes to github.com

Skip to content

use internal logging package where appropriate #4660

@haircommander

Description

@haircommander

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    good first issueDenotes an issue ready for a new contributor, according to the "help wanted" guidelines.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions