Skip to content

Commit 42be7a0

Browse files
authored
Do not 'lowercase' the path to crash/oome scripts. (#9496)
1 parent deb6fda commit 42be7a0

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

dd-java-agent/agent-crashtracking/src/main/java/datadog/crashtracking/Initializer.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -216,8 +216,7 @@ static String getScriptPathFromArg(String arg, String scriptNamePrefix) {
216216
if (arg == null || arg.isEmpty()) {
217217
return null;
218218
}
219-
arg = arg.toLowerCase();
220-
int idx = arg.indexOf(scriptNamePrefix);
219+
int idx = arg.toLowerCase().indexOf(scriptNamePrefix);
221220
if (idx < 0) {
222221
// the script name is not present in the value, so we cannot extract the path
223222
return null;

dd-java-agent/agent-crashtracking/src/test/java/datadog/crashtracking/ScriptInitializerTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ private static Stream<Arguments> crashTrackingScripts() {
8888
return Stream.of(
8989
Arguments.of("dd_crash_uploader.sh", ""),
9090
Arguments.of("dd_crash_uploader.bat", ""),
91-
Arguments.of("dd_crash_uploader.sh", " %p"),
91+
Arguments.of("dd_CrAsH_uploader.sh", " %p"),
9292
Arguments.of("dd_crash_uploader.bat", " %p"));
9393
}
9494

0 commit comments

Comments
 (0)