Closed CathalOConnorRH closed 2 years ago
enabling the uri metrics may lead to excessive data, it would be nice to be able to limit/filter the uri to specific keywords
added a URI_METRICS_FILTER_ENABLED flag to enable filtering and URI_METRICS_FILTER which uses a comma delimited list of keywords to filter.
Add an short answer for: Why it was done? (E.g The feature X was deprecated.)
Add an short answer for: How it was done? (E.g By removing this feature from ... OR By removing just the button but not its implementation ... )
Add the steps required to check this change. Following an example.
XX >> YY >> SS
N
X
PS.: Add images and/or .gifs to illustrate what was changed if this pull request modifies the appearance/output of something presented to the users.
@CathalOConnorRH it would be good to document those flags in the README
@pb82 Added flags to README and removed URI_METRICS_FILTER_ENABLED flag
Motivation
enabling the uri metrics may lead to excessive data, it would be nice to be able to limit/filter the uri to specific keywords
What
added a URI_METRICS_FILTER_ENABLED flag to enable filtering and URI_METRICS_FILTER which uses a comma delimited list of keywords to filter.
Why
Add an short answer for: Why it was done? (E.g The feature X was deprecated.)
How
Add an short answer for: How it was done? (E.g By removing this feature from ... OR By removing just the button but not its implementation ... )
Verification Steps
Add the steps required to check this change. Following an example.
XX >> YY >> SS
N
with the infoX
Checklist:
Progress
Additional Notes
PS.: Add images and/or .gifs to illustrate what was changed if this pull request modifies the appearance/output of something presented to the users.