Skip to content

Commit bb3292c

Browse files
committed
updated logging
1 parent 041cd40 commit bb3292c

6 files changed

Lines changed: 23 additions & 5 deletions

File tree

samples/scenarios/WorkItemFilteringSplitActivitiesJava/client/src/main/java/io/durabletask/samples/Client.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ public static void main(String[] args) throws Exception {
2626
String connectionString = ConnectionHelper.getConnectionString();
2727

2828
logger.info("=== Work Item Filtering Demo — Client ===");
29-
logger.info("Connection: {}", connectionString);
29+
logger.info("Connection: {}", ConnectionHelper.redact(connectionString));
3030

3131
// Create the Durable Task client
3232
DurableTaskClient client = DurableTaskSchedulerClientExtensions

samples/scenarios/WorkItemFilteringSplitActivitiesJava/infra/core/host/container-app.bicep

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ resource app 'Microsoft.App/containerApps@2025-01-01' = {
114114
tags: tags
115115
// It is critical that the identity is granted ACR pull access before the app is created
116116
// otherwise the container app will throw a provision error
117-
// This also forces us to use an user assigned managed identity since there would no way to
117+
// This also forces us to use a user assigned managed identity since there would no way to
118118
// provide the system assigned identity with the ACR pull access before the app is created
119119
dependsOn: usePrivateRegistry ? [ containerRegistryAccess ] : []
120120
identity: {

samples/scenarios/WorkItemFilteringSplitActivitiesJava/orchestrator-worker/src/main/java/io/durabletask/samples/OrchestratorWorker.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ public final class OrchestratorWorker {
2323
public static void main(String[] args) throws IOException, InterruptedException {
2424
String connectionString = ConnectionHelper.getConnectionString();
2525

26-
logger.info("[Orchestrator] Connection: {}", connectionString);
26+
logger.info("[Orchestrator] Connection: {}", ConnectionHelper.redact(connectionString));
2727
logger.info("[Orchestrator] This worker registers ONLY the orchestration. No activities.");
2828

2929
// Build the worker with only the orchestration registered.

samples/scenarios/WorkItemFilteringSplitActivitiesJava/shared/src/main/java/io/durabletask/samples/ConnectionHelper.java

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,24 @@ final class ConnectionHelper {
1717
private ConnectionHelper() {
1818
}
1919

20+
/**
21+
* Returns a redacted version of a connection string safe for logging.
22+
* Preserves Endpoint, TaskHub, and Authentication but redacts ClientID and other sensitive fields.
23+
*/
24+
static String redact(String connectionString) {
25+
if (connectionString == null) return "";
26+
StringBuilder sb = new StringBuilder();
27+
for (String part : connectionString.split(";")) {
28+
String key = part.split("=", 2)[0].trim();
29+
if (key.equalsIgnoreCase("Endpoint") || key.equalsIgnoreCase("TaskHub")
30+
|| key.equalsIgnoreCase("Authentication")) {
31+
if (sb.length() > 0) sb.append(';');
32+
sb.append(part);
33+
}
34+
}
35+
return sb.toString();
36+
}
37+
2038
static String getConnectionString() {
2139
String connectionString = System.getenv("DURABLE_TASK_CONNECTION_STRING");
2240
if (connectionString != null) {

samples/scenarios/WorkItemFilteringSplitActivitiesJava/shipper-worker/src/main/java/io/durabletask/samples/ShipperWorker.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ public final class ShipperWorker {
2424
public static void main(String[] args) throws IOException, InterruptedException {
2525
String connectionString = ConnectionHelper.getConnectionString();
2626

27-
logger.info("[Shipper] Connection: {}", connectionString);
27+
logger.info("[Shipper] Connection: {}", ConnectionHelper.redact(connectionString));
2828
logger.info("[Shipper] This worker registers ONLY the ShipOrder activity.");
2929

3030
// Build the worker with only the ShipOrder activity registered.

samples/scenarios/WorkItemFilteringSplitActivitiesJava/validator-worker/src/main/java/io/durabletask/samples/ValidatorWorker.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ public final class ValidatorWorker {
2323
public static void main(String[] args) throws IOException, InterruptedException {
2424
String connectionString = ConnectionHelper.getConnectionString();
2525

26-
logger.info("[Validator] Connection: {}", connectionString);
26+
logger.info("[Validator] Connection: {}", ConnectionHelper.redact(connectionString));
2727
logger.info("[Validator] This worker registers ONLY the ValidateOrder activity.");
2828

2929
// Build the worker with only the ValidateOrder activity registered.

0 commit comments

Comments
 (0)