Resolved build check issues (ex. spotlessCheck)

Author: kcsurapaneni

.fossa.yml

@@ -49,6 +49,9 @@ targets:
     - type: gradle
       path: ./
       target: ':testing:agent-for-testing'
+    - type: gradle
+      path: ./
+      target: ':instrumentation:activej-http:javaagent'
     - type: gradle
       path: ./
       target: ':instrumentation:alibaba-druid-1.0:javaagent'

instrumentation/activej-http/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/activejhttp/ActiveJHttpServerConnectionInstrumentation.java

@@ -1,3 +1,8 @@
+/*
+ * Copyright The OpenTelemetry Authors
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
 package io.opentelemetry.javaagent.instrumentation.activejhttp;
 
 import static io.opentelemetry.javaagent.bootstrap.Java8BytecodeBridge.currentContext;
@@ -25,10 +30,10 @@
 import net.bytebuddy.matcher.ElementMatcher;
 
 /**
- * <p>This class provides instrumentation for ActiveJ HTTP server connections by applying advice to the
- * {@code serve} method of classes that extend {@code io.activej.http.AsyncServlet}. The instrumentation is
- * designed to integrate with OpenTelemetry for distributed tracing, capturing and propagating trace context
- * through HTTP requests and responses.</p>
+ * This class provides instrumentation for ActiveJ HTTP server connections by applying advice to the
+ * {@code serve} method of classes that extend {@code io.activej.http.AsyncServlet}. The
+ * instrumentation is designed to integrate with OpenTelemetry for distributed tracing, capturing
+ * and propagating trace context through HTTP requests and responses.
  *
  * @author Krishna Chaitanya Surapaneni
  */
@@ -41,37 +46,39 @@ public class ActiveJHttpServerConnectionInstrumentation implements TypeInstrumen
    */
   @Override
   public ElementMatcher<TypeDescription> typeMatcher() {
-    return hasSuperType(named("io.activej.http.AsyncServlet"))
-        .and(not(isInterface()));
+    return hasSuperType(named("io.activej.http.AsyncServlet")).and(not(isInterface()));
   }
 
   /**
-   * Applies advice to the {@code serve} method of the matched classes. The advice captures trace context
-   * at the start of the method and propagates it through the response.
+   * Applies advice to the {@code serve} method of the matched classes. The advice captures trace
+   * context at the start of the method and propagates it through the response.
    *
    * @param transformer The {@code TypeTransformer} used to apply the advice.
    */
   @Override
   public void transform(TypeTransformer transformer) {
     transformer.applyAdviceToMethod(
-        isMethod().and(named("serve")).and(takesArguments(1)
-            .and(takesArgument(0, named("io.activej.http.HttpRequest")))),
+        isMethod()
+            .and(named("serve"))
+            .and(takesArguments(1).and(takesArgument(0, named("io.activej.http.HttpRequest")))),
         this.getClass().getName() + "$ServeAdvice");
   }
 
   /**
-   * <p>Inner class containing the advice logic for the {@code serve} method. This class defines two methods:</p>
+   * Inner class containing the advice logic for the {@code serve} method. This class defines two
+   * methods:
+   *
    * <ul>
-   *   <li>{@code methodEnter}: Captures the trace context at the start of the method.</li>
-   *   <li>{@code methodExit}: Propagates the trace context to the response and ends the span.</li>
+   *   <li>{@code methodEnter}: Captures the trace context at the start of the method.
+   *   <li>{@code methodExit}: Propagates the trace context to the response and ends the span.
    * </ul>
    */
   @SuppressWarnings("unused")
   public static class ServeAdvice {
 
     /**
-     * Advice executed at the start of the {@code serve} method. Captures the current trace context and
-     * starts a new span if tracing is enabled for the request.
+     * Advice executed at the start of the {@code serve} method. Captures the current trace context
+     * and starts a new span if tracing is enabled for the request.
      *
      * @param asyncServlet The {@code AsyncServlet} instance handling the request.
      * @param request The incoming HTTP request.
@@ -96,8 +103,8 @@ public static void methodEnter(
     }
 
     /**
-     * Advice executed at the end of the {@code serve} method. Propagates the trace context to the response,
-     * handles exceptions, and ends the span.
+     * Advice executed at the end of the {@code serve} method. Propagates the trace context to the
+     * response, handles exceptions, and ends the span.
      *
      * @param asyncServlet The {@code AsyncServlet} instance handling the request.
      * @param responsePromise The promise representing the HTTP response.
@@ -119,8 +126,8 @@ public static void methodExit(
       }
       String traceId = Span.fromContext(context).getSpanContext().getTraceId();
       String spanId = Span.fromContext(context).getSpanContext().getSpanId();
-      String traceFlags = Span.fromContext(context).getSpanContext().getTraceFlags().asHex()
-          .substring(0, 2);
+      String traceFlags =
+          Span.fromContext(context).getSpanContext().getTraceFlags().asHex().substring(0, 2);
       String traceparent = String.format("00-%s-%s-%s", traceId, spanId, traceFlags);
 
       scope.close();
@@ -135,24 +142,23 @@ public static void methodExit(
         instrumenter().end(context, httpRequest, httpResponse, error);
         responsePromise = Promise.of(httpResponse);
       } else if (throwable != null) {
-        HttpResponse httpResponse = HttpResponse.builder()
-            .withCode(500)
-            .withPlainText(throwable.getMessage())
-            .withHeader(HttpHeaders.of(traceparentHeader), traceparent)
-            .build();
-        instrumenter().end(context, httpRequest, httpResponse,
-            throwable);
+        HttpResponse httpResponse =
+            HttpResponse.builder()
+                .withCode(500)
+                .withPlainText(throwable.getMessage())
+                .withHeader(HttpHeaders.of(traceparentHeader), traceparent)
+                .build();
+        instrumenter().end(context, httpRequest, httpResponse, throwable);
         responsePromise = Promise.of(httpResponse);
         throwable = null;
       } else {
-        HttpResponse httpResponse = HttpResponse.notFound404()
-            .withHeader(HttpHeaders.of(traceparentHeader), traceparent)
-            .build();
-        instrumenter().end(context, httpRequest, httpResponse,
-            throwable);
+        HttpResponse httpResponse =
+            HttpResponse.notFound404()
+                .withHeader(HttpHeaders.of(traceparentHeader), traceparent)
+                .build();
+        instrumenter().end(context, httpRequest, httpResponse, throwable);
         responsePromise = Promise.of(httpResponse);
       }
     }
   }
-
 }

instrumentation/activej-http/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/activejhttp/ActiveJHttpServerConnectionInstrumentationModule.java

@@ -1,3 +1,8 @@
+/*
+ * Copyright The OpenTelemetry Authors
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
 package io.opentelemetry.javaagent.instrumentation.activejhttp;
 
 import static java.util.Collections.singletonList;
@@ -9,31 +14,34 @@
 import java.util.List;
 
 /**
- * <p>This class is an instrumentation module for ActiveJ HTTP server connections. It integrates with
- * OpenTelemetry to provide distributed tracing capabilities for applications using the ActiveJ HTTP server.</p>
+ * This class is an instrumentation module for ActiveJ HTTP server connections. It integrates with
+ * OpenTelemetry to provide distributed tracing capabilities for applications using the ActiveJ HTTP
+ * server.
  *
- * <p>The module is annotated with {@code @AutoService(InstrumentationModule.class)}, which automatically
- * registers it as a service provider for the OpenTelemetry instrumentation framework. This allows the module
- * to be discovered and loaded dynamically during runtime.</p>
+ * <p>The module is annotated with {@code @AutoService(InstrumentationModule.class)}, which
+ * automatically registers it as a service provider for the OpenTelemetry instrumentation framework.
+ * This allows the module to be discovered and loaded dynamically during runtime.
  *
  * @author Krishna Chaitanya Surapaneni
  */
 @AutoService(InstrumentationModule.class)
 public class ActiveJHttpServerConnectionInstrumentationModule extends InstrumentationModule {
 
   /**
-   * Constructs the instrumentation module with the specified instrumentation names. These names are used to
-   * identify the instrumentation in the OpenTelemetry framework.
+   * Constructs the instrumentation module with the specified instrumentation names. These names are
+   * used to identify the instrumentation in the OpenTelemetry framework.
    *
-   * <p>In this case, the module is identified by the names "activej-http" and "activej-http-server".</p>
+   * <p>In this case, the module is identified by the names "activej-http" and
+   * "activej-http-server".
    */
   public ActiveJHttpServerConnectionInstrumentationModule() {
     super("activej-http", "activej-http-server");
   }
 
   /**
-   * Returns a list of type instrumentation's provided by this module. Each type instrumentation applies advice
-   * to specific methods or classes to capture trace context and propagate it through HTTP requests and responses.
+   * Returns a list of type instrumentation's provided by this module. Each type instrumentation
+   * applies advice to specific methods or classes to capture trace context and propagate it through
+   * HTTP requests and responses.
    *
    * @return A list containing the {@code ActiveJHttpServerConnectionInstrumentation} instance.
    */
@@ -43,17 +51,16 @@ public List<TypeInstrumentation> typeInstrumentations() {
   }
 
   /**
-   * Registers helper resources required for the instrumentation. Helper resources are typically utility classes
-   * or configurations that support the instrumentation logic.
+   * Registers helper resources required for the instrumentation. Helper resources are typically
+   * utility classes or configurations that support the instrumentation logic.
    *
-   * <p>In this case, the {@code ActiveJHttpServerHelper} class is registered as a helper resource. This class
-   * provides utilities for creating HTTP responses with trace context.</p>
+   * <p>In this case, the {@code ActiveJHttpServerHelper} class is registered as a helper resource.
+   * This class provides utilities for creating HTTP responses with trace context.
    *
    * @param helperResourceBuilder The builder used to register helper resources.
    */
   @Override
   public void registerHelperResources(HelperResourceBuilder helperResourceBuilder) {
     helperResourceBuilder.register(ActiveJHttpServerHelper.class.getName());
   }
-
 }

instrumentation/activej-http/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/activejhttp/ActiveJHttpServerConnectionSingletons.java

@@ -1,3 +1,8 @@
+/*
+ * Copyright The OpenTelemetry Authors
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
 package io.opentelemetry.javaagent.instrumentation.activejhttp;
 
 import io.activej.http.HttpRequest;
@@ -6,9 +11,9 @@
 import io.opentelemetry.javaagent.bootstrap.internal.JavaagentHttpServerInstrumenters;
 
 /**
- * <p>This class provides singleton instances and utilities for ActiveJ HTTP server instrumentation. It is
- * designed to centralize the creation and access of OpenTelemetry-related components, such as the
- * {@code Instrumenter} used for tracing HTTP requests and responses.</p>
+ * This class provides singleton instances and utilities for ActiveJ HTTP server instrumentation. It
+ * is designed to centralize the creation and access of OpenTelemetry-related components, such as
+ * the {@code Instrumenter} used for tracing HTTP requests and responses.
  *
  * @author Krishna Chaitanya Surapaneni
  */

instrumentation/activej-http/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/activejhttp/ActiveJHttpServerHeaders.java

@@ -1,3 +1,8 @@
+/*
+ * Copyright The OpenTelemetry Authors
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
 package io.opentelemetry.javaagent.instrumentation.activejhttp;
 
 import io.activej.http.HttpHeader;
@@ -12,9 +17,9 @@
 import java.util.stream.Collectors;
 
 /**
- * <p>This enum implements the {@code ExtendedTextMapGetter<HttpRequest>} interface and provides methods for
- * extracting HTTP headers from an ActiveJ {@code HttpRequest}. It is used in the context of OpenTelemetry
- * instrumentation to propagate trace context across service boundaries.</p>
+ * This enum implements the {@code ExtendedTextMapGetter<HttpRequest>} interface and provides
+ * methods for extracting HTTP headers from an ActiveJ {@code HttpRequest}. It is used in the
+ * context of OpenTelemetry instrumentation to propagate trace context across service boundaries.
  *
  * @author Krishna Chaitanya Surapaneni
  */
@@ -48,5 +53,4 @@ public Iterator<String> getAll(HttpRequest carrier, String key) {
     }
     return values.iterator();
   }
-
 }