Skip to content

Update general API request received/completed logs from INFO to DEBUG#738

Merged
lornasong merged 1 commit intomainfrom
api-request-log-level
Mar 1, 2022
Merged

Update general API request received/completed logs from INFO to DEBUG#738
lornasong merged 1 commit intomainfrom
api-request-log-level

Conversation

@lornasong
Copy link
Member

These are logged for every API request and can clutter the logs

Use-case: currently Status API is used in Consul health checks, which can cause this message to log every 10s (or other configured interval)

These are logged for every API request and can clutter the logs
@lornasong lornasong requested review from a team and wilkermichael and removed request for a team February 28, 2022 20:55
@lornasong lornasong merged commit f7a70e4 into main Mar 1, 2022
@lornasong lornasong deleted the api-request-log-level branch March 1, 2022 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants