fix: Include user name in member creation log

relaycorp · Oct 10, 2023 · f5aa959 · f5aa959
1 parent a41b18a
commit f5aa959
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/member.spec.ts b/src/member.spec.ts
@@ -94,6 +94,9 @@ describe('member', () => {
       const result = await createMember(ORG_NAME, memberData, serviceOptions);
 
       expect(result.didSucceed).toBeTrue();
+      expect(mockLogging.logs).toContainEqual(
+        partialPinoLog('info', 'Member created', { userName: name }),
+      );
     });
 
     test('Missing name should be set to null', async () => {

diff --git a/src/member.ts b/src/member.ts
@@ -55,7 +55,10 @@ export async function createMember(
     throw err as Error;
   }
 
-  options.logger.info({ orgName, memberId: member.id }, 'Member created');
+  options.logger.info(
+    { orgName, userName: memberData.name, memberId: member.id },
+    'Member created',
+  );
   return {
     didSucceed: true,
     result: { id: member.id },