open-telemetry
/
opentelemetry-java-instrumentation
mirror of https://github.com/open-telemetry/opentelemetry-java-instrumentation.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Check for both regular and lowercase names with decorator enabled flags.

This commit is contained in:
Tyler Benson 2020-01-17 11:05:44 -08:00
parent 70c874b135
commit a280ae6b7f
2 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
dd-trace-api/src/main/java/datadog/trace/api/Config.java
View File

@ -591,7 +591,8 @@ public class Config {
}
public boolean isDecoratorEnabled(final String name) {
return getBooleanSettingFromEnvironment("trace." + name.toLowerCase() + ".enabled", true);
return getBooleanSettingFromEnvironment("trace." + name + ".enabled", true)
&& getBooleanSettingFromEnvironment("trace." + name.toLowerCase() + ".enabled", true);
}
/**

9
dd-trace-ot/src/test/groovy/datadog/opentracing/decorators/SpanDecoratorTest.groovy
View File

@ -457,7 +457,7 @@ class SpanDecoratorTest extends DDSpecification {
def "disable decorator via config"() {
setup:
ConfigUtils.updateConfig {
System.setProperty("dd.trace." + PeerServiceDecorator.getSimpleName().toLowerCase() + ".enabled", "false")
System.setProperty("dd.trace.${decorator}.enabled", "false")
}
tracer = DDTracer.builder()
@ -475,8 +475,13 @@ class SpanDecoratorTest extends DDSpecification {
cleanup:
ConfigUtils.updateConfig {
System.clearProperty("dd.trace." + PeerServiceDecorator.getSimpleName().toLowerCase() + ".enabled")
System.clearProperty("dd.trace.${decorator}.enabled")
}
where:
decorator | _
PeerServiceDecorator.getSimpleName().toLowerCase() | _
PeerServiceDecorator.getSimpleName() | _
}
def "disabling service decorator does not disable split by tags"() {