Skip to content

Replace logger middlware with zerolog one - #421

Merged
miry merged 1 commit into
masterfrom
http-api-server-timeout
Aug 28, 2022
Merged

Replace logger middlware with zerolog one#421
miry merged 1 commit into
masterfrom
http-api-server-timeout

Conversation

@miry

@miry miry commented Aug 28, 2022

Copy link
Copy Markdown
Contributor

Replace loggingMiddleware with zerolog/hlog one.

Example of new log record:

{"level":"debug","request_id":"cc5p6dfsemvo4p4a7sv0","client":"127.0.0.1:51587","method":"GET","url":"/version","status":200,"size":3,"duration":0.02825,"caller":"api.go:71","time":"2022-08-28T16:09:25Z"}

@miry miry self-assigned this Aug 28, 2022
@miry miry added the Toxiproxy label Aug 28, 2022
@miry miry added this to the 2.5.0 milestone Aug 28, 2022
@miry miry mentioned this pull request Aug 28, 2022
18 tasks
@miry
miry force-pushed the http-api-server-timeout branch 2 times, most recently from accde4f to 0e6e62f Compare August 28, 2022 16:09
@miry
miry marked this pull request as ready for review August 28, 2022 16:10
@miry
miry force-pushed the http-api-server-timeout branch 4 times, most recently from d234f6d to 11dee90 Compare August 28, 2022 16:27
Replace custom logger middleware with zerolog/hlog solutions.
Use logger instance from the HTTP request.
@miry
miry force-pushed the http-api-server-timeout branch from 11dee90 to 805fe34 Compare August 28, 2022 16:37
@miry
miry merged commit 805fe34 into master Aug 28, 2022
@miry
miry deleted the http-api-server-timeout branch August 28, 2022 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant