comment

Author: trask

instrumentation/spring/spring-web/spring-web-3.1/library/src/main/java/io/opentelemetry/instrumentation/spring/web/v3_1/internal/WebTelemetryUtil.java

@@ -33,6 +33,8 @@ public final class WebTelemetryUtil {
   @CanIgnoreReturnValue
   public static SpringWebTelemetryBuilder applyCommonConfig(
       SpringWebTelemetryBuilder builder, OpenTelemetry openTelemetry) {
+    // builderExtractor is guaranteed non-null because the builder class registers it during
+    // static initialization, before a builder instance can be passed here
     if (builderExtractor != null) {
       builderExtractor.apply(builder).configure(new CommonConfig(openTelemetry));
     }

instrumentation/spring/spring-webflux/spring-webflux-5.3/library/src/main/java/io/opentelemetry/instrumentation/spring/webflux/v5_3/internal/SpringWebfluxBuilderUtil.java

@@ -44,6 +44,8 @@ public final class SpringWebfluxBuilderUtil {
   @CanIgnoreReturnValue
   public static SpringWebfluxClientTelemetryBuilder applyClientCommonConfig(
       SpringWebfluxClientTelemetryBuilder builder, OpenTelemetry openTelemetry) {
+    // clientBuilderExtractor is guaranteed non-null because the builder class registers it during
+    // static initialization, before a builder instance can be passed here
     if (clientBuilderExtractor != null) {
       clientBuilderExtractor.apply(builder).configure(new CommonConfig(openTelemetry));
     }
@@ -53,6 +55,8 @@ public static SpringWebfluxClientTelemetryBuilder applyClientCommonConfig(
   @CanIgnoreReturnValue
   public static SpringWebfluxServerTelemetryBuilder applyServerCommonConfig(
       SpringWebfluxServerTelemetryBuilder builder, OpenTelemetry openTelemetry) {
+    // serverBuilderExtractor is guaranteed non-null because the builder class registers it during
+    // static initialization, before a builder instance can be passed here
     if (serverBuilderExtractor != null) {
       serverBuilderExtractor.apply(builder).configure(new CommonConfig(openTelemetry));
     }

instrumentation/spring/spring-webmvc/spring-webmvc-5.3/library/src/main/java/io/opentelemetry/instrumentation/spring/webmvc/v5_3/internal/SpringMvcBuilderUtil.java

@@ -35,6 +35,8 @@ public final class SpringMvcBuilderUtil {
   @CanIgnoreReturnValue
   public static SpringWebMvcTelemetryBuilder applyCommonConfig(
       SpringWebMvcTelemetryBuilder builder, OpenTelemetry openTelemetry) {
+    // builderExtractor is guaranteed non-null because the builder class registers it during
+    // static initialization, before a builder instance can be passed here
     if (builderExtractor != null) {
       builderExtractor.apply(builder).configure(new CommonConfig(openTelemetry));
     }

instrumentation/spring/spring-webmvc/spring-webmvc-6.0/library/src/main/java/io/opentelemetry/instrumentation/spring/webmvc/v6_0/internal/SpringMvcBuilderUtil.java

@@ -37,6 +37,8 @@ public final class SpringMvcBuilderUtil {
   @CanIgnoreReturnValue
   public static SpringWebMvcTelemetryBuilder applyCommonConfig(
       SpringWebMvcTelemetryBuilder builder, OpenTelemetry openTelemetry) {
+    // builderExtractor is guaranteed non-null because the builder class registers it during
+    // static initialization, before a builder instance can be passed here
     if (builderExtractor != null) {
       requireNonNull(builderExtractor).apply(builder).configure(new CommonConfig(openTelemetry));
     }