[ 
https://issues.apache.org/jira/browse/CXF-8424?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17300007#comment-17300007
 ] 

Andriy Redko commented on CXF-8424:
-----------------------------------

[~henriwi] the fix went into latest 3.3.10 / 3.4.3 / 3.5.0 SNAPSHOTs, could you 
please give it a try? Thank you!

> Use template URL instead of actual path values for JAX-RS micrometer metrics
> ----------------------------------------------------------------------------
>
>                 Key: CXF-8424
>                 URL: https://issues.apache.org/jira/browse/CXF-8424
>             Project: CXF
>          Issue Type: Improvement
>          Components: JAX-RS
>    Affects Versions: 3.4.1, 3.3.9
>            Reporter: Henrik Wingerei
>            Assignee: Andriy Redko
>            Priority: Minor
>             Fix For: 3.5.0, 3.4.3, 3.3.10
>
>
> The JAX-RS micrometer metrics implemented in the PRs listed below generates 
> one unique metric per combination of path parameteres in the URL. For 
> instance if you have the following path annotation on an API:
> {code:java}
> @Path("/user/{userId}" {code}
> A request to the URL _/user/1234_ will result in the following metric:
> {code:java}
> cxf_server_requests_seconds_count{exception="None",method="GET",operation="getUser",outcome="SUCCESS",status="200",uri="/user/1234",}
>  1.0 {code}
> Since the uri-tag in the metric uses the actual path value this could lead to 
> a huge number of metrics since it will create one metric per user. It will 
> also be more difficult to aggregate the metrics for one operation in tools 
> like Prometheus/Grafana.
> A suggestion is to instead generate the metric with the uri-tag as a 
> templated url. I think this is how the metrics generated by Spring MVC is 
> constructed. This would result in the following metric instead:
> {code:java}
> cxf_server_requests_seconds_count{exception="None",method="GET",operation="getUser",outcome="SUCCESS",status="200",uri="/user/{userId}",}
>  1.0  {code}
> Not sure if this is possible and what the best solution is, but it might be 
> possible to use the UriInfo class 
> ([https://javaee.github.io/javaee-spec/javadocs/javax/ws/rs/core/UriInfo.html)]
>   to get the URI without the values, or use that class to replace the actual 
> values with a placeholder/variable text.
> Relevant PRs:
> [https://github.com/apache/cxf/pull/716]
> [https://github.com/apache/cxf/pull/642] 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to