Fix Java warnings by suppressing deprecation warnings for backward compatibility

Author: Copilot

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/configuration/ConfigurationBuilder.java

@@ -104,7 +104,7 @@ public class ConfigurationBuilder {
   private static final String APPLICATIONINSIGHTS_PREVIEW_PROFILER_ENABLEDIAGNOSTICS =
       "APPLICATIONINSIGHTS_PREVIEW_PROFILER_ENABLEDIAGNOSTICS";
 
-  @Deprecated
+  // supported for backwards compatibility
   private static final String APPLICATIONINSIGHTS_PREVIEW_METRIC_INTERVAL_SECONDS =
       "APPLICATIONINSIGHTS_PREVIEW_METRIC_INTERVAL_SECONDS";
 
@@ -141,6 +141,8 @@ public static Configuration create(
     return config;
   }
 
+  @SuppressWarnings(
+      "deprecation") // logging warnings for usages of deprecated configuration options
   private static void logConfigurationWarnings(Configuration config) {
     if (config.instrumentation.micrometer.reportingIntervalSeconds != 60) {
       configurationLogger.warn(
@@ -354,6 +356,8 @@ private static void supportTelemetryProcessorsOldSemConv(Configuration config) {
     }
   }
 
+  @SuppressWarnings(
+      "deprecation") // support deprecated semconv attributes for backwards compatibility
   private static String mapAttributeKey(String oldAttributeKey) {
     String result = null;
     // Common attributes across HTTP client and server spans
@@ -479,6 +483,8 @@ private static boolean isOpenJ9Jvm() {
     return jvmName != null && jvmName.contains("OpenJ9");
   }
 
+  @SuppressWarnings(
+      "deprecation") // support deprecated configuration options for backwards compatibility
   private static void overlayAadEnvVars(
       Configuration config, Function<String, String> envVarsFunction) {
     String aadAuthString = getEnvVar(APPLICATIONINSIGHTS_AUTHENTICATION_STRING, envVarsFunction);

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/httpclient/LazyHttpClient.java

@@ -150,6 +150,8 @@ public Mono<HttpResponse> send(HttpRequest request, Context context) {
     return getDelegate().send(request, context);
   }
 
+  @SuppressWarnings(
+      "deprecation") // support deprecated configuration options for backwards compatibility
   private static HttpPipelinePolicy getAuthenticationPolicy(
       Configuration.AadAuthentication configuration, String aadAudienceWithScope) {
     switch (configuration.type) {
@@ -174,6 +176,7 @@ private static HttpPipelinePolicy getAuthenticationPolicyWithUami(
         managedIdentityCredential.build(), aadAudienceWithScope);
   }
 
+  @Deprecated
   private static HttpPipelinePolicy getAuthenticationPolicyWithClientSecret(
       Configuration.AadAuthentication configuration, String aadAudienceWithScope) {
     ClientSecretCredentialBuilder credential =

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/init/AiContextCustomizer.java

@@ -34,6 +34,8 @@ public AiContextCustomizer(
   }
 
   @Override
+  @SuppressWarnings(
+      "deprecation") // support deprecated semconv attributes for backwards compatibility
   public Context onStart(Context context, R request, Attributes startAttributes) {
 
     // TODO (trask) ideally would also check parentSpanContext !isValid || isRemote

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/processors/DelegatingLogData.java

@@ -6,6 +6,7 @@
 import static java.util.Objects.requireNonNull;
 
 import io.opentelemetry.api.common.Attributes;
+import io.opentelemetry.api.common.Value;
 import io.opentelemetry.api.logs.Severity;
 import io.opentelemetry.api.trace.SpanContext;
 import io.opentelemetry.sdk.common.InstrumentationScopeInfo;
@@ -59,10 +60,16 @@ public String getSeverityText() {
   }
 
   @Override
+  @SuppressWarnings("deprecation") // Implementation of deprecated method
   public Body getBody() {
     return delegate.getBody();
   }
 
+  @Override
+  public Value<?> getBodyValue() {
+    return delegate.getBodyValue();
+  }
+
   @Override
   public Attributes getAttributes() {
     return delegate.getAttributes();

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/processors/LogProcessor.java

@@ -10,7 +10,7 @@
 import io.opentelemetry.api.common.AttributeKey;
 import io.opentelemetry.api.common.Attributes;
 import io.opentelemetry.api.common.AttributesBuilder;
-import io.opentelemetry.sdk.logs.data.Body;
+import io.opentelemetry.api.common.Value;
 import io.opentelemetry.sdk.logs.data.LogRecordData;
 import java.util.ArrayList;
 import java.util.List;
@@ -84,7 +84,7 @@ public LogRecordData processFromAttributes(LogRecordData log) {
         updatedLogBuffer.setLength(updatedLogBuffer.length() - separator.length());
       }
 
-      return new MyLogData(log, existingLogAttributes, Body.string(updatedLogBuffer.toString()));
+      return new MyLogData(log, existingLogAttributes, Value.of(updatedLogBuffer.toString()));
     }
 
     return log;
@@ -107,7 +107,7 @@ public LogRecordData processToAttributes(LogRecordData log) {
           applyRule(groupNames.get(i), toAttributeRulePatterns.get(i), bodyAsString, builder);
     }
 
-    return new MyLogData(log, builder.build(), Body.string(bodyAsString));
+    return new MyLogData(log, builder.build(), Value.of(bodyAsString));
   }
 
   public static boolean logHasAllFromAttributeKeys(

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/processors/MyLogData.java

@@ -4,26 +4,26 @@
 package com.microsoft.applicationinsights.agent.internal.processors;
 
 import io.opentelemetry.api.common.Attributes;
-import io.opentelemetry.sdk.logs.data.Body;
+import io.opentelemetry.api.common.Value;
 import io.opentelemetry.sdk.logs.data.LogRecordData;
 
 public class MyLogData extends DelegatingLogData {
 
   private final Attributes attributes;
-  private final Body body;
+  private final Value<?> body;
 
   public MyLogData(LogRecordData delegate, Attributes attributes) {
-    this(delegate, attributes, delegate.getBody());
+    this(delegate, attributes, delegate.getBodyValue());
   }
 
-  public MyLogData(LogRecordData delegate, Attributes attributes, Body body) {
+  public MyLogData(LogRecordData delegate, Attributes attributes, Value<?> body) {
     super(delegate);
     this.attributes = attributes;
     this.body = body;
   }
 
   @Override
-  public Body getBody() {
+  public Value<?> getBodyValue() {
     return body;
   }
 

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/profiler/service/ServiceProfilerClient.java

@@ -4,6 +4,7 @@
 package com.microsoft.applicationinsights.agent.internal.profiler.service;
 
 import com.azure.core.exception.HttpResponseException;
+import com.azure.core.http.HttpHeaderName;
 import com.azure.core.http.HttpMethod;
 import com.azure.core.http.HttpPipeline;
 import com.azure.core.http.HttpRequest;
@@ -69,7 +70,7 @@ private static BlobAccessPass getUploadAccess(HttpResponse response) {
       if (response.getStatusCode() >= 300) {
         throw new HttpResponseException(response);
       }
-      String location = response.getHeaderValue("Location");
+      String location = response.getHeaderValue(HttpHeaderName.LOCATION);
       if (location == null || location.isEmpty()) {
         throw new AssertionError("response did not have a location");
       }
@@ -86,7 +87,7 @@ public Mono<HttpResponse> executePostWithRedirect(URL requestUrl) {
 
     HttpRequest request = new HttpRequest(HttpMethod.POST, requestUrl);
     if (userAgent != null) {
-      request.setHeader("User-Agent", userAgent);
+      request.setHeader(HttpHeaderName.USER_AGENT, userAgent);
     }
     return httpPipeline.send(request);
   }

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/sampling/SamplingOverrides.java

@@ -122,6 +122,8 @@ private StrictMatcher(String key, String value) {
     }
 
     @Override
+    @SuppressWarnings(
+        "deprecation") // support deprecated semconv attributes for backwards compatibility
     public boolean test(
         Attributes attributes, LazyHttpUrl lazyHttpUrl, LazyHttpTarget lazyHttpTarget) {
       String val = MatcherGroup.getValueIncludingThreadName(attributes, key);
@@ -213,6 +215,8 @@ private KeyOnlyMatcher(String key) {
     }
 
     @Override
+    @SuppressWarnings(
+        "deprecation") // support deprecated semconv attributes for backwards compatibility
     public boolean test(
         Attributes attributes,
         @Nullable LazyHttpUrl lazyHttpUrl,
@@ -247,6 +251,8 @@ private String get() {
     }
   }
 
+  @SuppressWarnings(
+      "deprecation") // support deprecated semconv attributes for backwards compatibility
   private static boolean getHttpUrlKeyOldOrStableSemconv(AttributeKey<String> key) {
     String keyString = key.getKey();
     return keyString.equals(HttpIncubatingAttributes.HTTP_URL.getKey())