Skip to content
This repository has been archived by the owner on Apr 19, 2023. It is now read-only.

Mixpanel events imported with incorrect time #88

Open
derekbekoe opened this issue Sep 25, 2021 · 3 comments
Open

Mixpanel events imported with incorrect time #88

derekbekoe opened this issue Sep 25, 2021 · 3 comments
Labels
wontfix This will not be worked on

Comments

@derekbekoe
Copy link

In /server/lib/senders/mixpanel.js#L69, the time of the event is set as follows

const now = Date.now();
const mixpanelEvents = logs.map((log) => {
const eventName = loggingTools.logTypes.get(log.type);
log.time = now;

This is incorrect as this is the time the job running, not the time of the Auth0 log event.

The expectation is that the time of the Auth0 log event is what is sent to mixpanel.

Here is an example of the incorrect result in Mixpanel.
It shows all the events happened in the same minute but in reality, the Auth0 log events happened several minutes apart.

Screen Shot 2021-09-25 at 16 30 46

The correct datetime is sent to mixpanel in the "date" field so this fal

Screen Shot 2021-09-25 at 16 34 09

Proposed fix:

Update


to

log.time = Date.parse(log.date) || now;
@stale
Copy link

stale bot commented Jan 9, 2022

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

@stale stale bot added the wontfix This will not be worked on label Jan 9, 2022
@derekbekoe
Copy link
Author

This is still an issue with the auth0 logs provider. Blocked on waiting for a response from the maintainers.

@stale stale bot removed the wontfix This will not be worked on label Jan 10, 2022
@stale
Copy link

stale bot commented Apr 16, 2022

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

@stale stale bot added the wontfix This will not be worked on label Apr 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant