Skip to content

Commit 587421b

Browse files
Removed PII logging from GraphGroupOwnerReader
1 parent 4478a1f commit 587421b

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

Service/GroupMembershipManagement/Repositories.GraphGroups/GraphGroupOwnerReader.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@ public async Task<bool> IsEmailRecipientOwnerOfGroupAsync(string userIdentifier,
154154
await _loggingRepository.LogMessageAsync(new LogMessage
155155
{
156156
RunId = runId,
157-
Message = $"Checking on user existence for user identifier '{userIdentifier}' to determine if it is an owner of group {groupObjectId}."
157+
Message = $"Checking on user existence for user identifier to determine if it is an owner of group {groupObjectId}."
158158
});
159159

160160
Guid userId;
@@ -166,7 +166,7 @@ await _loggingRepository.LogMessageAsync(new LogMessage
166166
await _loggingRepository.LogMessageAsync(new LogMessage
167167
{
168168
RunId = runId,
169-
Message = $"Getting user information for email '{userIdentifier}'."
169+
Message = $"Getting user information for email."
170170
});
171171

172172
User user = null;
@@ -193,7 +193,7 @@ await _graphServiceClient.Users.GetAsync(requestConfiguration =>
193193
await _loggingRepository.LogMessageAsync(new LogMessage
194194
{
195195
RunId = runId,
196-
Message = $"No user was found when checking for user with email {userIdentifier}."
196+
Message = $"No user was found when checking for user with email."
197197
});
198198

199199
return false;
@@ -224,7 +224,7 @@ await _loggingRepository.LogMessageAsync(new LogMessage
224224
await _loggingRepository.LogMessageAsync(new LogMessage
225225
{
226226
RunId = runId,
227-
Message = $"User identifier is already an object id: '{userId}'."
227+
Message = $"User identifier is already an object id."
228228
});
229229
}
230230

0 commit comments

Comments
 (0)