Skip to content

Add observability to asset-worker outer entrypoint RPC methods#13474

Draft
courtney-sims wants to merge 2 commits intocloudflare:mainfrom
courtney-sims:csims/wc-4482-2
Draft

Add observability to asset-worker outer entrypoint RPC methods#13474
courtney-sims wants to merge 2 commits intocloudflare:mainfrom
courtney-sims:csims/wc-4482-2

Conversation

@courtney-sims
Copy link
Copy Markdown
Contributor

@courtney-sims courtney-sims commented Apr 15, 2026

Fixes #[insert GH or internal issue link(s)].

Describe your change...

The outer entrypoint's RPC methods (unstable_canFetch, unstable_getByETag, unstable_getByPathname, unstable_exists) previously had no error reporting or tracing. That was only configured for the main fetch. When the router-worker called the RPC methods and they threw, the error was invisible.
This was discovered when deploying the loopback changes to staging. Both flamingo edge tests failed, but nothing appeared in Sentry.

  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because: observability changes only
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: internal changes only

A picture of a cute animal (not mandatory, but encouraged)

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 15, 2026

⚠️ No Changeset found

Latest commit: a80ca4f

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.

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

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Apr 15, 2026

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@13474

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@13474

miniflare

npm i https://pkg.pr.new/miniflare@13474

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@13474

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@13474

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@13474

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@13474

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@13474

wrangler

npm i https://pkg.pr.new/wrangler@13474

commit: a80ca4f

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

Labels

None yet

Projects

Status: Untriaged

Development

Successfully merging this pull request may close these issues.

2 participants