chore(api): log /api/v1/healthz requests at debug level instead of info to reduce noise #481
Conversation
…fo to reduce noise
|
@obazoud is attempting to deploy a commit to the Hookdeck Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Pull Request Overview
This PR reduces logging noise by changing the log level for health check requests from info to debug. This is a common practice to avoid cluttering logs with frequent health check calls that don't provide operational value.
- Changes logging level for
/api/v1/healthzrequests from info to debug
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| if c.Request.URL.Path == "/api/v1/healthz" { | ||
| logger.Debug("healthz request completed", fields...) | ||
| } else { | ||
| logger.Info("request completed", fields...) | ||
| } |
There was a problem hiding this comment.
The indentation is inconsistent with the rest of the codebase. The if-else block should use tabs to match the surrounding code formatting.
| if c.Request.URL.Path == "/api/v1/healthz" { | |
| logger.Debug("healthz request completed", fields...) | |
| } else { | |
| logger.Info("request completed", fields...) | |
| } | |
| if c.Request.URL.Path == "/api/v1/healthz" { | |
| logger.Debug("healthz request completed", fields...) | |
| } else { | |
| logger.Info("request completed", fields...) | |
| } |
Copilot uses AI. Check for mistakes.
| } | ||
| } else { | ||
| logger.Info("request completed", fields...) | ||
| if c.Request.URL.Path == "/api/v1/healthz" { |
There was a problem hiding this comment.
Small nit, would it be better to check "string ending with /healthz and starting with /api" so it works for new versions in the future.
Not a big deal tho, just let me know, we can merge as-is.
No description provided.