Skip to content

Commit

Permalink
Add method and host to uriMapper
Browse files Browse the repository at this point in the history
Cardinality won't become a problem? Right? ...right?
  • Loading branch information
NovaFox161 committed Apr 18, 2024
1 parent 0d6339b commit ce841be
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ class BeanConfig {
fun httpClient(registry: ObservationRegistry): OkHttpClient {
val interceptor = OkHttpObservationInterceptor.builder(registry, "okhttp.requests")
// This can lead to tag cardinality explosion as it doesn't use uri patterns, should investigate options for that one day
.uriMapper { it.url.encodedPath }
.uriMapper { "${it.method} ${it.url.host}${it.url.encodedPath}" }
.build()

return OkHttpClient.Builder()
Expand Down

0 comments on commit ce841be

Please sign in to comment.