Skip to content

Commit dfa3953

Browse files
authored
Merge branch 'master' into ci/update-gradle-dependencies-20250615
2 parents 500f4ea + 88aa5b2 commit dfa3953

5 files changed

Lines changed: 19 additions & 4 deletions

File tree

dd-java-agent/instrumentation/org-json/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,5 +33,5 @@ dependencies {
3333
testRuntimeOnly project(':dd-java-agent:instrumentation:iast-instrumenter')
3434
testRuntimeOnly project(':dd-java-agent:instrumentation:java-io') //Needed for Reader
3535

36-
latestDepTestImplementation group: 'org.json', name: 'json', version: '+'
36+
latestDepTestImplementation group: 'org.json', name: 'json', version: '20250107' // TODO replace with + when APPSEC-58007 solved
3737
}

dd-trace-api/src/main/java/datadog/trace/api/ConfigDefaults.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ public final class ConfigDefaults {
113113
static final int DEFAULT_APPSEC_TRACE_RATE_LIMIT = 100;
114114
static final boolean DEFAULT_APPSEC_WAF_METRICS = true;
115115
static final int DEFAULT_APPSEC_WAF_TIMEOUT = 100000; // 0.1 s
116-
static final boolean DEFAULT_API_SECURITY_ENABLED = false;
116+
static final boolean DEFAULT_API_SECURITY_ENABLED = true;
117117
static final float DEFAULT_API_SECURITY_SAMPLE_DELAY = 30.0f;
118118
// TODO: change to true once the RFC is approved
119119
static final boolean DEFAULT_API_SECURITY_ENDPOINT_COLLECTION_ENABLED = false;

dd-trace-api/src/main/java/datadog/trace/api/config/GeneralConfig.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,8 @@ public final class GeneralConfig {
5454

5555
public static final String RUNTIME_METRICS_ENABLED = "runtime.metrics.enabled";
5656
public static final String RUNTIME_ID_ENABLED = "runtime-id.enabled";
57+
public static final String RUNTIME_METRICS_RUNTIME_ID_ENABLED =
58+
"runtime.metrics.runtime-id.enabled";
5759

5860
public static final String HEALTH_METRICS_ENABLED = "trace.health.metrics.enabled";
5961
public static final String HEALTH_METRICS_STATSD_HOST = "trace.health.metrics.statsd.host";

internal-api/src/main/java/datadog/trace/api/Config.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -601,7 +601,8 @@ private Config(final ConfigProvider configProvider, final InstrumenterConfig ins
601601
this.configProvider = configProvider;
602602
this.instrumenterConfig = instrumenterConfig;
603603
configFileStatus = configProvider.getConfigFileStatus();
604-
runtimeIdEnabled = configProvider.getBoolean(RUNTIME_ID_ENABLED, true);
604+
runtimeIdEnabled =
605+
configProvider.getBoolean(RUNTIME_ID_ENABLED, true, RUNTIME_METRICS_RUNTIME_ID_ENABLED);
605606
runtimeVersion = System.getProperty("java.version", "unknown");
606607

607608
// Note: We do not want APiKey to be loaded from property for security reasons

internal-api/src/test/groovy/datadog/trace/api/ConfigForkedTest.groovy

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,18 +3,30 @@ package datadog.trace.api
33
import spock.lang.Specification
44

55
import static datadog.trace.api.config.GeneralConfig.RUNTIME_ID_ENABLED
6+
import static datadog.trace.api.config.GeneralConfig.RUNTIME_METRICS_RUNTIME_ID_ENABLED
67

78
class ConfigForkedTest extends Specification {
89

910
static final String PREFIX = "dd."
1011

12+
def getSetting() {
13+
RUNTIME_ID_ENABLED
14+
}
15+
1116
def "test random runtime id generation can be turned off"(){
1217
setup:
13-
System.setProperty(PREFIX + RUNTIME_ID_ENABLED, "false")
18+
System.setProperty(PREFIX + getSetting(), "false")
1419

1520
when:
1621
def config = new Config()
1722
then:
1823
config.runtimeId == ""
1924
}
2025
}
26+
27+
class RuntimeMetricsRuntimeIdAliasForkedTest extends ConfigForkedTest {
28+
@Override
29+
def getSetting() {
30+
RUNTIME_METRICS_RUNTIME_ID_ENABLED
31+
}
32+
}

0 commit comments

Comments
 (0)