diff --git a/src/services/tracking/hooks.spec.js b/src/services/tracking/hooks.spec.js index 4b7002a066..11260b30b9 100644 --- a/src/services/tracking/hooks.spec.js +++ b/src/services/tracking/hooks.spec.js @@ -107,6 +107,7 @@ describe('useTracking', () => { expect(Sentry.setUser).toHaveBeenCalledWith({ email: 'fake@test.com', username: 'eugene_onegin', + ip_address: '{{auto}}', }) ) }) @@ -145,11 +146,13 @@ describe('useTracking', () => { setup({ me: user }) }) - it('sets null user in sentry', async () => { + it('sets user in sentry', async () => { renderHook(() => useTracking(), { wrapper }) await waitFor(() => expect(Sentry.setUser).toHaveBeenCalled()) - await waitFor(() => expect(Sentry.setUser).toHaveBeenCalledWith(null)) + await waitFor(() => + expect(Sentry.setUser).toHaveBeenCalledWith({ ip_address: '{{auto}}' }) + ) }) }) @@ -165,7 +168,9 @@ describe('useTracking', () => { renderHook(() => useTracking(), { wrapper }) await waitFor(() => expect(Sentry.setUser).toHaveBeenCalled()) - await waitFor(() => expect(Sentry.setUser).toHaveBeenCalledWith(null)) + await waitFor(() => + expect(Sentry.setUser).toHaveBeenCalledWith({ ip_address: '{{auto}}' }) + ) }) }) }) diff --git a/src/services/tracking/useTracking.js b/src/services/tracking/useTracking.js index acf0eb70d7..e785a97ff6 100644 --- a/src/services/tracking/useTracking.js +++ b/src/services/tracking/useTracking.js @@ -42,16 +42,17 @@ export function useTracking() { useTrackFeatureFlags(user) // TODO: Can probably delete useUpdatePendoWithOwner(user) - const maybeSentryUser = {} + const sentryUser = {} if (user?.email) { - maybeSentryUser.email = user?.email + sentryUser.email = user?.email } if (user?.user?.username) { - maybeSentryUser.username = user?.user?.username + sentryUser.username = user?.user?.username } - const sentryUser = - Object.keys(maybeSentryUser).length === 0 ? null : maybeSentryUser + // https://docs.sentry.io/platforms/javascript/enriching-events/identify-user/#ip_address + // eslint-disable-next-line + sentryUser.ip_address = '{{auto}}' Sentry.setUser(sentryUser) return { data: user, ...all }