[UNOMI-522] EventCollector saving temporary profiles

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: unomi-1.4.0, unomi-1.5.7
• Fix Version/s: unomi-2.0.0
• Component/s: unomi(-core)
• Assignee: Kevan Jahanshahi
• Reporter: Kevan Jahanshahi
• Created: 27/Oct/21 16:12
• Updated: 05/May/23 12:34
• Resolved: 11/Aug/22 08:47

Description

There is issues with profile handling in the EventCollector.

• In case you send an event that will update the profile, like a login event.
• on a new session ID
• with no profile already known by unomi
• EventCollector is creating a temporary profile, prefixed by: "temp_"
• normally, this kind of profile should not persisted, but in case the event sent trigger rules and actions that will modify the profile (that is new), then the profile will be saved with the temporary id prefixed by "temp_"

I think the profile creation handling in the event collector is really bad, and should no deal with temporary id.

The question is not easy to answer, does the event collector should do the same as context.js ? 

• create the profile
• send the profile id in the cookie ?

May be event collector should not be used to create profile, but then how can we control this ?

Anyway we should not persist this temporary profile.

It can be dangerous with the profile merge, because user can get this temporary profile assigned to them.