feat(platform-api): include user-agent in access logs - #3277
feat(platform-api): include user-agent in access logs#3277renuka-fernando wants to merge 1 commit into
Conversation
|
Warning Review limit reached
Next review available in: 4 minutes Limit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?Wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Replace gin.Default() with gin.New() plus explicit logger and recovery middleware so the access log can be customized. Add a custom formatter that mirrors gin's default format and appends the request User-Agent for client identification. Signed-off-by: Renuka Fernando <renukapiyumal@gmail.com>
33e8121 to
049aeff
Compare
|
@coderabbitai review |
|
Purpose
The platform-api access log is produced by gin's built-in logger, whose format is fixed and does not capture the client User-Agent, making it hard to identify which clients are calling the API.
This PR extends the access log to include the request User-Agent.
Approach
gin.Default()withgin.New()plus explicit logger and recovery middleware so the access-log format can be customized.Related Issues
N/A
Checklist