From 895ffdbb13d682c84317b78aaa6ab9cf849f7a8c Mon Sep 17 00:00:00 2001
From: NThakur4 <nthakur4@slb.com>
Date: Tue, 14 Feb 2023 14:18:40 -0600
Subject: [PATCH] add user info on error
---
.../azure/util/SubscriberNotificationRequestLogger.java | 2 +-
.../util/SubscriberNotificationRequestLoggerTest.java | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/provider/notification-azure/src/main/java/org/opengroup/osdu/notification/provider/azure/util/SubscriberNotificationRequestLogger.java b/provider/notification-azure/src/main/java/org/opengroup/osdu/notification/provider/azure/util/SubscriberNotificationRequestLogger.java
index 9cad07742..e05e8f485 100644
--- a/provider/notification-azure/src/main/java/org/opengroup/osdu/notification/provider/azure/util/SubscriberNotificationRequestLogger.java
+++ b/provider/notification-azure/src/main/java/org/opengroup/osdu/notification/provider/azure/util/SubscriberNotificationRequestLogger.java
@@ -29,7 +29,7 @@ public class SubscriberNotificationRequestLogger implements ISubscriberNotificat
.build();
final String data = response.isSuccessCode()
? String.format("notificationId=%s subscriptionId=%s", notificationId, subscription.getId())
- : String.format("notificationId=%s subscriptionId=%s response=%s", notificationId, subscription.getId(), response.getBody());
+ : String.format("notificationId=%s subscriptionId=%s createdBy=%s response=%s", notificationId, subscription.getId(), subscription.getCreatedBy(), response.getBody());
loggingOptions.setData(data);
this.dependencyLogger.logDependency(loggingOptions);
}
diff --git a/provider/notification-azure/src/test/java/org/opengroup/osdu/notification/util/SubscriberNotificationRequestLoggerTest.java b/provider/notification-azure/src/test/java/org/opengroup/osdu/notification/util/SubscriberNotificationRequestLoggerTest.java
index 4f5d5d5d9..d8ecdba19 100644
--- a/provider/notification-azure/src/test/java/org/opengroup/osdu/notification/util/SubscriberNotificationRequestLoggerTest.java
+++ b/provider/notification-azure/src/test/java/org/opengroup/osdu/notification/util/SubscriberNotificationRequestLoggerTest.java
@@ -68,7 +68,7 @@ public class SubscriberNotificationRequestLoggerTest {
ArgumentCaptor<DependencyLoggingOptions> loggingOptionsArgumentCaptor = ArgumentCaptor.forClass(DependencyLoggingOptions.class);
verify(dependencyLogger, times(1)).logDependency(loggingOptionsArgumentCaptor.capture());
DependencyLoggingOptions actualLoggingOptions = loggingOptionsArgumentCaptor.getValue();
- verifyDependencyLogging(actualLoggingOptions, "notificationId=test-notification-id subscriptionId=id_1 response=unauthorized", "http://challenge", 403, false);
+ verifyDependencyLogging(actualLoggingOptions, "notificationId=test-notification-id subscriptionId=id_1 createdBy=test@test.com response=unauthorized", "http://challenge", 403, false);
}
private void verifyDependencyLogging(DependencyLoggingOptions capturedLoggingOptions, String data, String target, int resultCode, boolean success) {
--
GitLab