Skip to content

chore(error-tracking): support cookieless#60145

Merged
z0br0wn merged 2 commits into
masterfrom
zbrown/error-tracking-cookieless
May 27, 2026
Merged

chore(error-tracking): support cookieless#60145
z0br0wn merged 2 commits into
masterfrom
zbrown/error-tracking-cookieless

Conversation

@z0br0wn
Copy link
Copy Markdown
Contributor

@z0br0wn z0br0wn commented May 26, 2026

Problem

error tracking doesn't support cookieless processing, which it should (to keep in line with the other pipelines).

Changes

  • wire up cookieless processing (this is already brought in on the charts side via the common config)
  • add the step to the pipeline
  • also incorporate the split in overflow steps to do pre and post cookieless

@z0br0wn z0br0wn requested a review from a team May 26, 2026 19:44
@assign-reviewers-posthog assign-reviewers-posthog Bot requested a review from a team May 26, 2026 19:45
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 26, 2026

Reviews (1): Last reviewed commit: "chore(error-tracking): support cookieles..." | Re-trigger Greptile

Copy link
Copy Markdown
Contributor

@pl pl left a comment

Choose a reason for hiding this comment

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

LGTM, one minor comment.

Comment thread nodejs/src/ingestion/cookieless/cookieless-manager.ts
Copy link
Copy Markdown
Member

@lricoy lricoy left a comment

Choose a reason for hiding this comment

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

lgtm, fyi we're removing the "stateless" cookieless mode: #51311 but I don't think this affects anything on this PR.

Thanks for doing it

@z0br0wn z0br0wn merged commit df8777d into master May 27, 2026
144 checks passed
@z0br0wn z0br0wn deleted the zbrown/error-tracking-cookieless branch May 27, 2026 18:10
@deployment-status-posthog
Copy link
Copy Markdown

deployment-status-posthog Bot commented May 27, 2026

Deploy status

Environment Status Deployed At Workflow
dev ✅ Deployed 2026-05-27 18:40 UTC Run
prod-us ✅ Deployed 2026-05-27 19:01 UTC Run
prod-eu ✅ Deployed 2026-05-27 19:04 UTC Run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants