chore: remove audit user since it cannot be shown in ui

Author: purplecabbage

src/commands/app/deploy.js

@@ -26,7 +26,7 @@ const {
 const rtLib = require('@adobe/aio-lib-runtime')
 const LogForwarding = require('../../lib/log-forwarding')
 const { sendAppAssetsDeployedAuditLog, sendAppDeployAuditLog } = require('../../lib/audit-logger')
-const { setRuntimeApiHostAndAuthHandler, getAccessToken, getTokenData } = require('../../lib/auth-helper')
+const { setRuntimeApiHostAndAuthHandler, getAccessToken } = require('../../lib/auth-helper')
 const logActions = require('../../lib/log-actions')
 
 const PRE_DEPLOY_EVENT_REG = 'pre-deploy-event-reg'
@@ -68,8 +68,6 @@ class Deploy extends BuildCommand {
 
       if (cliDetails?.accessToken) {
         try {
-          // store user id from token data for cdn deploy audit metadata
-          appInfo.auditUserId = getTokenData(cliDetails.accessToken)?.user_id
           // send audit log at start (don't wait for deployment to finish)
           await sendAppDeployAuditLog({
             accessToken: cliDetails?.accessToken,
@@ -132,10 +130,7 @@ class Deploy extends BuildCommand {
       // - break into smaller pieces deploy, allowing to first deploy all actions then all web assets
       for (let i = 0; i < keys.length; ++i) {
         const k = keys[i]
-        // auditUserId is only set if it is available in the token data
-        // falsy because "", 0, false, null, undefined, NaN, etc. are all invalid values
         const v = {
-          ...(appInfo.auditUserId && { auditUserId: appInfo.auditUserId }),
           ...setRuntimeApiHostAndAuthHandler(values[i])
         }
         await this.deploySingleConfig({ name: k, config: v, originalConfig: values[i], flags, spinner })

test/commands/app/deploy.test.js

@@ -198,9 +198,7 @@ beforeEach(() => {
       env: 'stage'
     }
   })
-  authHelper.getTokenData.mockImplementation(() => {
-    return null // default to null, tests can override
-  })
+
   LogForwarding.init.mockResolvedValue(mockLogForwarding)
 
   command = new TheCommand([])
@@ -673,136 +671,6 @@ describe('run', () => {
     expect(open).toHaveBeenCalledWith('http://prefix?fake=https://example.com')
   })
 
-  test('deploy should pass auditUserId to deployWeb config when user_id is present in token', async () => {
-    const mockUserId = 'test-user-123'
-    const mockToken = 'mock.token.value'
-
-    authHelper.getAccessToken.mockResolvedValueOnce({
-      accessToken: mockToken,
-      env: 'stage'
-    })
-    authHelper.getTokenData.mockReturnValueOnce({
-      user_id: mockUserId
-    })
-
-    command.getAppExtConfigs.mockResolvedValueOnce(createAppConfig(command.appConfig))
-    mockWebLib.deployWeb.mockResolvedValue('https://example.com')
-
-    command.argv = []
-    await command.run()
-
-    expect(command.error).toHaveBeenCalledTimes(0)
-    expect(authHelper.getTokenData).toHaveBeenCalledWith(mockToken)
-    expect(mockWebLib.deployWeb).toHaveBeenCalledWith(
-      expect.objectContaining({
-        auditUserId: mockUserId
-      }),
-      expect.any(Function)
-    )
-  })
-
-  test('deploy should NOT include auditUserId in config when user_id is undefined', async () => {
-    const mockToken = 'mock.token.value'
-
-    authHelper.getAccessToken.mockResolvedValueOnce({
-      accessToken: mockToken,
-      env: 'stage'
-    })
-    authHelper.getTokenData.mockReturnValueOnce({
-      // user_id is undefined
-    })
-
-    command.getAppExtConfigs.mockResolvedValueOnce(createAppConfig(command.appConfig))
-    mockWebLib.deployWeb.mockResolvedValue('https://example.com')
-
-    command.argv = []
-    await command.run()
-
-    expect(command.error).toHaveBeenCalledTimes(0)
-    expect(mockWebLib.deployWeb).toHaveBeenCalledWith(
-      expect.not.objectContaining({
-        auditUserId: expect.anything()
-      }),
-      expect.any(Function)
-    )
-  })
-
-  test('deploy should NOT include auditUserId in config when user_id is null', async () => {
-    const mockToken = 'mock.token.value'
-
-    authHelper.getAccessToken.mockResolvedValueOnce({
-      accessToken: mockToken,
-      env: 'stage'
-    })
-    authHelper.getTokenData.mockReturnValueOnce({
-      user_id: null
-    })
-
-    command.getAppExtConfigs.mockResolvedValueOnce(createAppConfig(command.appConfig))
-    mockWebLib.deployWeb.mockResolvedValue('https://example.com')
-
-    command.argv = []
-    await command.run()
-
-    expect(command.error).toHaveBeenCalledTimes(0)
-    expect(mockWebLib.deployWeb).toHaveBeenCalledWith(
-      expect.not.objectContaining({
-        auditUserId: expect.anything()
-      }),
-      expect.any(Function)
-    )
-  })
-
-  test('deploy should NOT include auditUserId in config when user_id is empty string', async () => {
-    const mockToken = 'mock.token.value'
-
-    authHelper.getAccessToken.mockResolvedValueOnce({
-      accessToken: mockToken,
-      env: 'stage'
-    })
-    authHelper.getTokenData.mockReturnValueOnce({
-      user_id: ''
-    })
-
-    command.getAppExtConfigs.mockResolvedValueOnce(createAppConfig(command.appConfig))
-    mockWebLib.deployWeb.mockResolvedValue('https://example.com')
-
-    command.argv = []
-    await command.run()
-
-    expect(command.error).toHaveBeenCalledTimes(0)
-    expect(mockWebLib.deployWeb).toHaveBeenCalledWith(
-      expect.not.objectContaining({
-        auditUserId: expect.anything()
-      }),
-      expect.any(Function)
-    )
-  })
-
-  test('deploy should NOT include auditUserId when getTokenData returns null', async () => {
-    const mockToken = 'mock.token.value'
-
-    authHelper.getAccessToken.mockResolvedValueOnce({
-      accessToken: mockToken,
-      env: 'stage'
-    })
-    authHelper.getTokenData.mockReturnValueOnce(null)
-
-    command.getAppExtConfigs.mockResolvedValueOnce(createAppConfig(command.appConfig))
-    mockWebLib.deployWeb.mockResolvedValue('https://example.com')
-
-    command.argv = []
-    await command.run()
-
-    expect(command.error).toHaveBeenCalledTimes(0)
-    expect(mockWebLib.deployWeb).toHaveBeenCalledWith(
-      expect.not.objectContaining({
-        auditUserId: expect.anything()
-      }),
-      expect.any(Function)
-    )
-  })
-
   test('deploy should show action urls (web-export: true)', async () => {
     command.getAppExtConfigs.mockResolvedValueOnce(createAppConfig(command.appConfig))
     mockRuntimeLib.deployActions.mockResolvedValue({

test/commands/lib/auth-helper.test.js

@@ -61,14 +61,6 @@ describe('getTokenData', () => {
   beforeEach(() => {
     jest.clearAllMocks()
   })
-
-  test('should call through to getImsTokenData to decode JWT token and return payload', () => {
-    getImsTokenData.mockReturnValue({ user_id: '12345', name: 'Test User' })
-    // Example JWT token with payload: {"user_id":"12345","name":"Test User"}
-    const exampleToken = 'eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJ1c2VyX2lkIjoiMTIzNDUiLCJuYW1lIjoiVGVzdCBVc2VyIn0.sflKxwRJSMeKKF2QT4fwpMeJf36POk6yJV_adQssw5c'
-    const result = getTokenData(exampleToken)
-    expect(result).toEqual({ user_id: '12345', name: 'Test User' })
-  })
   test('should return null for invalid token', () => {
     getImsTokenData.mockImplementation(() => { throw new Error('Invalid token') })
     const invalidToken = 'invalid.token.string'