search

Netflix / dgs-framework

GraphQL for Java with Spring Boot made easy.
https://netflix.github.io/dgs
Apache License 2.0
3.03k stars 286 forks source link

Fix DGS Micrometer timer configuration #1799

Closed kilink closed 5 months ago

kilink commented 5 months ago

A recent update broke the documented autotime-related properties that live under "management.metrics.dgs-graphql.autotime", since it renamed the property on the DgsGraphQLMetricsProperties class from "autoTime" to "autoTimeProperties". Fix the naming to restore the old config prefix, and add a test case that covers it.

Additionally, move the AutoTimer / PropertiesAutoTimer out of DgsGraphQLMetricsProperties, and make it a dependency of DgsGraphQLMetricsInstrumentation and the other classes that depend on it.

Bug was introduced in 21e625a; this PR fixes #1249.