Skip to content

fix(server): stop sending client timestamps - #218

Closed
feugy wants to merge 1 commit into
canaryfrom
damienfeugas/exp-3628-stop-sending-ts
Closed

feugy wants to merge 1 commit into
canaryfrom
damienfeugas/exp-3628-stop-sending-ts

Conversation

@feugy

@feugy feugy commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

🖖 What's in there?

Part 4 of EXP-3628: Web Analytics events get their timestamp at ingestion time, not from the client.

The server-side track(), trackExposure(), identify() and group() functions stop sending ts. Vercel's ingestion service sets the timestamp when it receives the event.

🤺 How to test?

  • pnpm -F @vercel/analytics test: the request body checks in src/server/*.test.ts no longer expect ts.

🔬 Notes to reviewers

  • ⚠️ Merge after vercel/api#98450 is deployed. The ingestion service requires ts on the current production version, so it would reject these requests with 400.
  • Older versions of the package still send ts. The ingestion service accepts it and ignores it.

🔗 Related PRs

All PRs are independent (not stacked). Deploy them in this order:

  1. vercel/api#98449: the subscriber reads the ingestedAt SNS attribute.
  2. vercel/api#98450: ingestion sends ingestedAt and ignores client ts.
  3. vercel/front#90985: the injected script stops sending ts.
  4. This PR: server-side functions stop sending ts.

@vercel

vercel Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
analytics-astro Ready Ready Preview Oct 16, 2026 8:07am UTC
analytics-next15 Ready Ready Preview Oct 16, 2026 8:07am UTC
analytics-nextjs Ready Ready Preview Oct 16, 2026 8:07am UTC
analytics-nuxt Ready Ready Preview Oct 16, 2026 8:07am UTC
analytics-remix Ready Ready Preview Oct 16, 2026 8:07am UTC
analytics-sveltekit Ready Ready Preview Oct 16, 2026 8:07am UTC
analytics-sveltekit-old Ready Ready Preview Oct 16, 2026 8:07am UTC
analytics-vue Ready Ready Preview Oct 16, 2026 8:07am UTC

Request Review

Vercel's ingestion service sets the event timestamp at ingestion time and ignores ts. (EXP-3628)

This branch was successfully deployed

8 active deployments
Preview – analytics-nuxt — 0f2512a4 Deployed Oct 16, 2026 by vercel[bot]
Preview – analytics-nextjs — 0f2512a4 Deployed Oct 16, 2026 by vercel[bot]
Preview – analytics-sveltekit — 0f2512a4 Deployed Oct 16, 2026 by vercel[bot]
Preview – analytics-next15 — 0f2512a4 Deployed Oct 16, 2026 by vercel[bot]
Preview – analytics-sveltekit-old — 0f2512a4 Deployed Oct 16, 2026 by vercel[bot]
Preview – analytics-remix — 0f2512a4 Deployed Oct 16, 2026 by vercel[bot]
Preview – analytics-vue — 0f2512a4 Deployed Oct 16, 2026 by vercel[bot]
Preview – analytics-astro — 0f2512a4 Deployed Oct 16, 2026 by vercel[bot]
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.

1 participant