Fix applicationinsights.json config is not loaded in manual rp attach…

… scenario (#3288)
microsoft · Sep 14, 2023 · 98a5e0a · 98a5e0a
1 parent 9128b61
commit 98a5e0a
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/.../com/microsoft/applicationinsights/agent/internal/configuration/ConfigurationBuilder.java b/.../com/microsoft/applicationinsights/agent/internal/configuration/ConfigurationBuilder.java
@@ -3,6 +3,7 @@
 
 package com.microsoft.applicationinsights.agent.internal.configuration;
 
+import com.azure.monitor.opentelemetry.exporter.implementation.statsbeat.RpAttachType;
 import com.azure.monitor.opentelemetry.exporter.implementation.utils.HostName;
 import com.azure.monitor.opentelemetry.exporter.implementation.utils.Strings;
 import com.fasterxml.jackson.core.JsonParseException;
@@ -478,7 +479,8 @@ private static Configuration loadConfigurationFile(Path agentJarPath) {
       return getConfiguration(runtimeAttachedConfigurationContent, JsonOrigin.RUNTIME_ATTACHED);
     }
 
-    if (SdkVersionPrefixHolder.isRpIntegration()) {
+    // only RP auto integrations do not support loading applicationinsights.json
+    if (RpAttachType.getRpAttachType() == RpAttachType.INTEGRATED_AUTO) {
       // users do not have write access to agent directory in rp integrations
       // and rp integrations should not use applicationinsights.json because that makes it difficult
       // to merge rp intent and user intent