Do not include system-properties in test session command tag (#7187)

DataDog · Jun 14, 2024 · ecd93fb · ecd93fb
1 parent 33a2dcd
commit ecd93fb
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 32 deletions.
diff --git a/...adle/src/main/groovy/datadog/trace/instrumentation/gradle/CiVisibilityGradleListener.java b/...adle/src/main/groovy/datadog/trace/instrumentation/gradle/CiVisibilityGradleListener.java
@@ -151,15 +151,6 @@ private static String recreateStartCommand(StartParameter startParameter, String
       }
     }
 
-    for (Map.Entry<String, String> e : startParameter.getSystemPropertiesArgs().entrySet()) {
-      String propertyKey = e.getKey();
-      String propertyValue = e.getValue();
-      command.append(" -D").append(propertyKey);
-      if (propertyValue != null && !propertyValue.isEmpty()) {
-        command.append('=').append(propertyValue);
-      }
-    }
-
     return command.toString();
   }
 

diff --git a/...ion/gradle/src/main/groovy/datadog/trace/instrumentation/gradle/legacy/GradleUtils.groovy b/...ion/gradle/src/main/groovy/datadog/trace/instrumentation/gradle/legacy/GradleUtils.groovy
@@ -49,15 +49,6 @@ abstract class GradleUtils {
       }
     }
 
-    for (Map.Entry<String, String> e : startParameter.getSystemPropertiesArgs().entrySet()) {
-      String propertyKey = e.getKey()
-      String propertyValue = e.getValue()
-      command.append(" -D").append(propertyKey)
-      if (propertyValue != null && !propertyValue.isEmpty()) {
-        command.append('=').append(propertyValue)
-      }
-    }
-
     return command.toString()
   }
 

diff --git a/...n-3.2.1/src/main/java/datadog/trace/instrumentation/maven3/MavenLifecycleParticipant.java b/...n-3.2.1/src/main/java/datadog/trace/instrumentation/maven3/MavenLifecycleParticipant.java
@@ -104,7 +104,7 @@ public void afterProjectsRead(MavenSession session) {
         // otherwise reference to "@{argLine}" that we add when configuring tracer
         // might cause failure
         // (test executions config is changed even if auto configuration is disabled:
-        // for passing module and sesion IDs to child JVM)
+        // for passing module and session IDs to child JVM)
         projectProperties.setProperty("argLine", "");
       }
     }

diff --git a/...umentation/maven-3.2.1/src/main/java/datadog/trace/instrumentation/maven3/MavenUtils.java b/...umentation/maven-3.2.1/src/main/java/datadog/trace/instrumentation/maven3/MavenUtils.java
@@ -4,7 +4,6 @@
 import java.lang.invoke.MethodHandle;
 import java.util.Iterator;
 import java.util.List;
-import java.util.Map;
 import java.util.Objects;
 import java.util.Properties;
 import org.apache.maven.BuildFailureException;
@@ -164,18 +163,6 @@ public static String getCommandLine(MavenSession session) {
       command.append(' ').append(goal);
     }
 
-    Properties userProperties = request.getUserProperties();
-    if (userProperties != null) {
-      for (Map.Entry<Object, Object> e : userProperties.entrySet()) {
-        command
-            .append(" -")
-            .append(CLIManager.SET_SYSTEM_PROPERTY)
-            .append(e.getKey())
-            .append('=')
-            .append(e.getValue());
-      }
-    }
-
     if (!request.getActiveProfiles().isEmpty()) {
       command.append(" -").append(CLIManager.ACTIVATE_PROFILES);
       Iterator<String> it = request.getActiveProfiles().iterator();