Skip to content

Add client_session context entity if anonymous tracking with session tracking is enabled (close #1124) - #1135

Merged
Matus Tomlein (matus-tomlein) merged 3 commits into
release/3.9.0-beta.1from
issue/1124-client_session
Jan 16, 2023
Merged

Add client_session context entity if anonymous tracking with session tracking is enabled (close #1124)#1135
Matus Tomlein (matus-tomlein) merged 3 commits into
release/3.9.0-beta.1from
issue/1124-client_session

Conversation

@matus-tomlein

Copy link
Copy Markdown
Contributor

Issue #1124

Adds the client_session context entity to events even in case anonymous tracking with session tracking is enabled. That means with this configuration:

snowplow('newTracker', 'sp', '{{collector_url_here}}', {
  ...
  anonymousTracking: { withSessionTracking: true },
  contexts: { session: true }
);

In that case, the userId and previousSessionId properties of the entity are anonymised (we do the same on the mobile trackers). The userId is assigned a zero UUID (00000000-0000-0000-0000-000000000000) while the previousSessionId is null.

@bundlemon

bundlemon Bot commented Dec 23, 2022

Copy link
Copy Markdown

BundleMon

Files added (6)
Status Path Size Limits
libraries/browser-tracker-core/dist/index.mod
ule.js
+23.85KB 25KB / +10%
trackers/javascript-tracker/dist/sp.js
+23.5KB 25KB / +10%
trackers/javascript-tracker/dist/sp.lite.js
+14.21KB 15KB / +10%
trackers/browser-tracker/dist/index.umd.min.j
s
+14.06KB 15KB / +10%
libraries/tracker-core/dist/index.module.js
+13.35KB 15KB / +10%
trackers/browser-tracker/dist/index.module.js
+3.46KB 5KB / +10%

Total files change +92.44KB 0%

Final result: ✅

View report in BundleMon website ➡️


Current branch size history

Base automatically changed from release/3.8.0 to master January 3, 2023 15:34
});
});

function createTracker(configuration?: TrackerConfiguration) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@matus-tomlein
Matus Tomlein (matus-tomlein) changed the base branch from master to release/3.9.0-beta.1 January 16, 2023 10:56
@matus-tomlein
Matus Tomlein (matus-tomlein) merged commit 9ba6cf2 into release/3.9.0-beta.1 Jan 16, 2023
@matus-tomlein
Matus Tomlein (matus-tomlein) deleted the issue/1124-client_session branch January 16, 2023 10:57
Matus Tomlein (matus-tomlein) added a commit that referenced this pull request Jan 17, 2023
Peter Perlepes (igneel64) pushed a commit that referenced this pull request Feb 8, 2023
Peter Perlepes (igneel64) pushed a commit that referenced this pull request Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants