Skip to content

Commit c1eba03

Browse files
Copilottrask
andcommitted
Debug configuration mapping and investigate attribute inclusion issue
Co-authored-by: trask <[email protected]>
1 parent c229a7d commit c1eba03

File tree

3 files changed

+9
-1
lines changed

3 files changed

+9
-1
lines changed

agent/agent-tooling/src/main/java/com/microsoft/applicationinsights/agent/internal/init/AiConfigCustomizer.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -236,6 +236,7 @@ private static void enableInstrumentations(
236236
}
237237
if (config.instrumentation.jdbc.captureQueryParameters) {
238238
properties.put("otel.instrumentation.jdbc.capture-query-parameters", "true");
239+
System.out.println("DEBUG: Setting otel.instrumentation.jdbc.capture-query-parameters=true");
239240
}
240241
}
241242
if (config.instrumentation.jms.enabled) {

smoke-tests/apps/Jdbc/src/smokeTest/java/com/microsoft/applicationinsights/smoketest/JdbcQueryParametersTest.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,10 @@ void hsqldbPreparedStatementCapturesQueryParameters() throws Exception {
2929
assertThat(telemetry.rdd1.getData()).isEqualTo("select * from abc where xyz = ?");
3030
assertThat(telemetry.rdd1.getType()).isEqualTo("SQL");
3131
assertThat(telemetry.rdd1.getTarget()).isEqualTo("hsqldb | testdb");
32+
33+
// Debug: print all properties to see what we actually have
34+
System.out.println("DEBUG: RDD properties: " + telemetry.rdd1.getProperties());
35+
3236
// Query parameters should be captured when captureQueryParameters is enabled
3337
assertThat(telemetry.rdd1.getProperties()).containsEntry("db.query.parameter.0", "y");
3438
assertThat(telemetry.rdd1.getSuccess()).isTrue();

smoke-tests/apps/Jdbc/src/smokeTest/resources/query_parameters_applicationinsights.json

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,10 @@
88
},
99
"instrumentation": {
1010
"jdbc": {
11-
"captureQueryParameters": true
11+
"captureQueryParameters": true,
12+
"masking": {
13+
"enabled": false
14+
}
1215
}
1316
}
1417
}

0 commit comments

Comments
 (0)