You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Oct 18, 2022. It is now read-only.
we're currently using the GelfLoggingFilter to log access requests. Since our loadbalancers send HTTP pings to a special endpoint, this creates a lot of noise, that we don't need in our logs.
I would love to extend the current GelfLoggingFilter implementation by adding an optional filtering parameter to the GelfLoggingFilter constructor that will be called in the logRequest method, when we have access to both the response and the request and drop such (matching an url and response code 200) access messages.
To make this as extensible as possible, I would suggest the usage of a functional interface that receives all the parameters that the receives the last four parameters that the logRequest method gets.
Would you accept such an addition? If so, I will prepare a MR.
Thanks
The text was updated successfully, but these errors were encountered:
Hi there,
we're currently using the GelfLoggingFilter to log access requests. Since our loadbalancers send HTTP pings to a special endpoint, this creates a lot of noise, that we don't need in our logs.
I would love to extend the current GelfLoggingFilter implementation by adding an optional filtering parameter to the GelfLoggingFilter constructor that will be called in the logRequest method, when we have access to both the response and the request and drop such (matching an url and response code 200) access messages.
To make this as extensible as possible, I would suggest the usage of a functional interface that receives all the parameters that the receives the last four parameters that the logRequest method gets.
Would you accept such an addition? If so, I will prepare a MR.
Thanks
The text was updated successfully, but these errors were encountered: