Skip to content

Commit 8b8ba20

Browse files
Merge branch 'master' into alexeyk/groovy-4
# Conflicts: # dd-java-agent/instrumentation/spring/spring-cloud-zuul-2.0/build.gradle # dd-java-agent/instrumentation/spring/spring-data-1.8/build.gradle # dd-java-agent/instrumentation/spring/spring-webmvc/spring-webmvc-3.1/build.gradle # dd-java-agent/instrumentation/spring/spring-webmvc/spring-webmvc-5.3/build.gradle # dd-java-agent/instrumentation/spring/spring-webmvc/spring-webmvc-6.0/build.gradle
2 parents 6a4b28a + 614cc72 commit 8b8ba20

File tree

3 files changed

+10
-14
lines changed

3 files changed

+10
-14
lines changed

dd-java-agent/instrumentation/spring/spring-cloud-zuul-2.0/build.gradle

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ tasks.named("compileJava", JavaCompile) {
2525
}
2626

2727
groovySpock {
28-
configureDependencies("testImplementation")
2928
configureDependencies("testImplementation", libs.spock.spring.groovy4)
3029
}
3130

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

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -96,11 +96,12 @@ static Ledger ledger(int size) {
9696

9797
/**
9898
* Deprecated in favor of typed getters like...
99+
*
99100
* <ul>
100-
* <li>{@link TagMap#getObject(String)}
101-
* <li>{@link TagMap#getString(String)}
102-
* <li>{@link TagMap#getBoolean(String)
103-
* <li>...
101+
* <li>{@link TagMap#getObject(String)}
102+
* <li>{@link TagMap#getString(String)}
103+
* <li>{@link TagMap#getBoolean(String)}
104+
* <li>...
104105
* </ul>
105106
*/
106107
@Deprecated

utils/config-utils/src/main/java/datadog/trace/bootstrap/config/provider/StableConfigSource.java

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -28,16 +28,14 @@ public final class StableConfigSource extends ConfigProvider.Source {
2828
private final StableConfig config;
2929

3030
StableConfigSource(String filePath, ConfigOrigin origin) {
31+
StableConfig cfg = StableConfig.EMPTY;
3132
this.fileOrigin = origin;
32-
File file = new File(filePath);
33-
if (!file.exists()) {
34-
this.config = StableConfig.EMPTY;
35-
return;
36-
}
37-
StableConfig cfg;
3833
try {
34+
File file = new File(filePath);
3935
log.debug("Stable configuration file found at path: {}", file);
40-
cfg = StableConfigParser.parse(filePath);
36+
if (file.exists()) {
37+
cfg = StableConfigParser.parse(filePath);
38+
}
4139
} catch (Throwable e) {
4240
if (e instanceof StableConfigMappingException
4341
|| e instanceof IllegalArgumentException
@@ -55,8 +53,6 @@ public final class StableConfigSource extends ConfigProvider.Source {
5553
filePath,
5654
e.getMessage());
5755
}
58-
59-
cfg = StableConfig.EMPTY;
6056
}
6157
this.config = cfg;
6258
}

0 commit comments

Comments
 (0)