-
Notifications
You must be signed in to change notification settings - Fork 504
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support configuration of access log format #294
Comments
Do we have any plan on this ? May be parsing a param from route down to Finagle that will help ? |
It's not on the short-term roadmap AFAIK. Are there specific formats you want to support? |
It would be great to include the time taken to serve the request (like the %{UNIT}T flag in apache's format strings) |
@bashofmann good idea! FYI, you can get aggregate statistics of that info through the metrics.json endpoint currently (and raw data in individual traces). |
We currently hard-code the format for access logs; it would be better if we allowed this to be configured. It probably makes the most sense to support Apache's format strings.
The text was updated successfully, but these errors were encountered: