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

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

Thanks for checking [~henriwi] , I think this was the last issue holding off 
the vote for release, so I hope it is going to happen shortly. With respect to 
Exception / 500 : the CXF default behavior is propagate the exception and fault 
interceptors are not called (I do not know why it was done this way to be 
fair). Anyway, you could set the busproperty:
{code:java}
bus.setProperty("org.apache.cxf.propagate.exception", Boolean.FALSE); {code}
to disabled that. It should register all metrics, the sample test case is there 
[1]. Thank you.

[1] 
https://github.com/apache/cxf/blob/master/systests/spring-boot/src/test/java/org/apache/cxf/systest/jaxrs/spring/boot/SpringJaxrsTest.java#L93

> 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