-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
Path Normalization #3
Comments
Another option could be to use Route (api/cities/{id}) instead of a Path (api/cities/2342) and avoid using RegExes if possible. This way it'll be also easier to detect wildcard routes "/api/search/{*searchString}". What do you think? |
I will write prototype for test. And then I'll think about how best to work with it. |
Any progress with this? I'm trying to get metrics for request durations that also include paths, but it would be a lot more useful if there is a way to normalize ID values. |
@dmeenhuis I will come back to this issue soon. Thank you! |
Thanks! I managed to fork this repo and incorporate the changes in the linked PR and for now that seems to work fine, although it contains a minor bug for the integer normalization (it doesn't properly deal with integer values at the end of the path, where there is no trailing slash). Would love to see an update from your end, thanks for this very useful library so far 👍 |
@dmeenhuis Is it me or you didn't push or create a PR? I can't find it in this repo |
It's not you, I didn't actually push the changes yet. Can do that tomorrow if you like. |
I finished work with this:prom-client-net/prom-client#18 Now, I can come back to this issue |
Released. 1.1.0 |
Regex examples |
If something does not work, then reopen. |
Add 'Path Normalization' with regex.
Dirty version: https://github.com/PrometheusClientNet/Prometheus.Client.HttpRequestDurations/pull/1/files
The text was updated successfully, but these errors were encountered: