From 4c5a3e075bfc93b9f239b80cc7d834c9983f4c94 Mon Sep 17 00:00:00 2001 From: Brandon Sara Date: Wed, 7 Jun 2023 17:00:18 -0600 Subject: [PATCH] fix: updated MetricsInstrumentation to implement correct instrumentExecutionResult method --- .../web/servlet/metrics/MetricsInstrumentation.java | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/graphql-spring-boot-autoconfigure/src/main/java/graphql/kickstart/autoconfigure/web/servlet/metrics/MetricsInstrumentation.java b/graphql-spring-boot-autoconfigure/src/main/java/graphql/kickstart/autoconfigure/web/servlet/metrics/MetricsInstrumentation.java index 37062d7c..86d37a20 100644 --- a/graphql-spring-boot-autoconfigure/src/main/java/graphql/kickstart/autoconfigure/web/servlet/metrics/MetricsInstrumentation.java +++ b/graphql-spring-boot-autoconfigure/src/main/java/graphql/kickstart/autoconfigure/web/servlet/metrics/MetricsInstrumentation.java @@ -1,6 +1,7 @@ package graphql.kickstart.autoconfigure.web.servlet.metrics; import graphql.ExecutionResult; +import graphql.execution.instrumentation.InstrumentationState; import graphql.execution.instrumentation.parameters.InstrumentationExecutionParameters; import graphql.execution.instrumentation.tracing.TracingInstrumentation; import io.micrometer.core.instrument.MeterRegistry; @@ -39,6 +40,12 @@ public MetricsInstrumentation(MeterRegistry meterRegistry, boolean tracingEnable @Override public CompletableFuture instrumentExecutionResult( ExecutionResult executionResult, InstrumentationExecutionParameters parameters) { + return this.instrumentExecutionResult(executionResult, parameters, null); + } + + @Override + public CompletableFuture instrumentExecutionResult( + ExecutionResult executionResult, InstrumentationExecutionParameters parameters, InstrumentationState rawState) { if (executionResult.getExtensions() != null && executionResult.getExtensions().containsKey(TRACING)) {