Skip to content

fix(telemetry): route the collector domain and redirect browser visits - #695

Merged
jumski merged 1 commit into
mainfrom
fix/telemetry-domain-and-docs-redirect
Oct 2, 2026
Merged

jumski merged 1 commit into
mainfrom
fix/telemetry-domain-and-docs-redirect

Conversation

@jumski

@jumski jumski commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Redirect browser GET requests to https://pgflow.dev/reference/telemetry/ with HTTP 302; leave POST telemetry ingestion unchanged.
  • Keep other HTTP methods at 405 and check that redirects/rejections do not write Analytics Engine points.
  • Declare telemetry.pgflow.dev as the collector's Worker Custom Domain so Wrangler manages DNS and TLS at deployment.

Checks

  • pnpm nx test-env:fresh edge-worker passed.
  • pnpm nx run @pgflow/telemetry-worker:test passed: 32 tests.
  • Nx reports only @pgflow/telemetry-worker affected.
  • pnpm nx affected -t test typecheck lint --base=HEAD passed before commit.
  • pnpm --dir apps/telemetry-worker exec wrangler deploy --dry-run passed with the existing PGFLOW_TELEMETRY Analytics Engine binding.
  • git diff --check passed; the committed tree matches the checked files.

Deployment

This PR does not deploy Cloudflare infrastructure. After merge, run the collector deployment from main with a token authorized for the Worker and Custom Domain. Keep request logging disabled. GET redirects to the docs; a valid JSON POST returns 204. A production smoke-test POST writes a real Analytics Engine point.

No npm/JSR release or changeset is needed: the collector is private and 0.17.2 senders already use this hostname.

Related: #691/#692 corrected the sender endpoint; this PR declares the hosting route and browser behavior.

Browser GET requests previously returned 405. Redirect them to the telemetry reference with 302 while preserving POST ingestion and rejecting other methods without Analytics Engine writes.

Declare telemetry.pgflow.dev as a Worker Custom Domain so Wrangler manages DNS and TLS during deployment. Collector tests, typecheck, lint, and the Wrangler deployment dry-run pass; no live infrastructure changes or package release are part of this commit.
@changeset-bot

changeset-bot Bot commented Oct 2, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: b562139

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

jumski commented Oct 2, 2026

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@nx-cloud

nx-cloud Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit b562139

Command Status Duration Result
nx run-many -t build --projects=dsl,core,cli ✅ Succeeded 5s View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-10-02 07:38:49 UTC

jumski commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor Author

Merge activity

  • Oct 2, 7:40 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Oct 2, 7:40 AM UTC: @jumski merged this pull request with Graphite.

@jumski
jumski merged commit 9160db6 into main Oct 2, 2026
17 checks passed
@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

🚀 Production Deployment: Website

✅ Successfully deployed to production!

🔗 Production URL: https://pgflow.dev

📝 Details:

  • Commit: 9160db62009f9535264824a7e76593c2e09b1769
  • View Logs

Deployed at: 2026-10-02T09:40:22+02:00

This branch was successfully deployed

1 active deployment
preview — b562139e Deployed Oct 2, 2026 by jumski via deploy-website #3416
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