Skip to content

feat(cloudflare): Add span ops for rate limit and Durable Object WebSocket spans - #23211

Open
msonnb wants to merge 1 commit into
developfrom
ms/cloudflare-rate-limit-websocket-span-ops
Open

feat(cloudflare): Add span ops for rate limit and Durable Object WebSocket spans#23211
msonnb wants to merge 1 commit into
developfrom
ms/cloudflare-rate-limit-websocket-span-ops

Conversation

@msonnb

@msonnb msonnb commented Aug 10, 2026

Copy link
Copy Markdown
Member

Rate limit binding calls now emit the rpc op with rpc.service, and the Durable Object webSocketMessage/webSocketClose/webSocketError handlers emit the websocket op. Both sets of spans previously had no op at all.

part of #23138

…ocket spans

Rate limit binding calls now emit the `rpc` op with `rpc.service`, and the
Durable Object `webSocketMessage`/`webSocketClose`/`webSocketError` handlers
emit the `websocket` op. Both sets of spans previously had no op at all.

Ref JS-3318

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

size-limit report 📦

⚠️ Warning: Base artifact is not the latest one, because the latest workflow run is not done yet. This may lead to incorrect results. Try to re-run all tests to get up to date results.

Path Size % Change Change
@sentry/browser 30.35 kB - -
@sentry/browser - with treeshaking flags 28.52 kB - -
@sentry/browser - with treeshaking flags tracing without tracing 26.85 kB - -
@sentry/browser (incl. Tracing) 48.63 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 48.64 kB - -
@sentry/browser (incl. Tracing, Profiling) 53.48 kB - -
@sentry/browser (incl. Tracing, Replay) 88.06 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 77.47 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 92.78 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 105.47 kB - -
@sentry/browser (incl. Feedback) 47.69 kB - -
@sentry/browser (incl. sendFeedback) 35.18 kB - -
@sentry/browser (incl. FeedbackAsync) 40.33 kB - -
@sentry/browser (incl. Metrics) 31.43 kB - -
@sentry/browser (incl. Logs) 31.66 kB - -
@sentry/browser (incl. Metrics & Logs) 32.34 kB - -
@sentry/react 32.14 kB - -
@sentry/react (incl. Tracing) 50.83 kB - -
@sentry/vue 35.45 kB - -
@sentry/vue (incl. Tracing) 50.6 kB - -
@sentry/svelte 30.38 kB - -
CDN Bundle 32.32 kB - -
CDN Bundle (incl. Tracing) 49.6 kB - -
CDN Bundle (incl. Logs, Metrics) 33.84 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 50.9 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 74.38 kB - -
CDN Bundle (incl. Tracing, Replay) 87.13 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 88.41 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 92.94 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 94.19 kB - -
CDN Bundle - uncompressed 95.83 kB - -
CDN Bundle (incl. Tracing) - uncompressed 148.84 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 100.42 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 152.82 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 229.34 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 268.09 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 272.06 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 281.79 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 285.74 kB - -
@sentry/nextjs (client) 53.41 kB - -
@sentry/sveltekit (client) 49.04 kB - -
@sentry/core/server 65.7 kB - -
@sentry/core/browser 52.02 kB - -
@sentry/node 119.23 kB - -
@sentry/node/import (ESM hook with diagnostics-channel injection) 0 B added added
@sentry/node - without tracing 83.23 kB +0.01% +1 B 🔺
@sentry/aws-serverless 92.51 kB - -
@sentry/cloudflare (withSentry) - minified 215.03 kB +0.06% +116 B 🔺
@sentry/cloudflare (withSentry) 530.97 kB +0.07% +350 B 🔺

View base workflow run

@msonnb
msonnb marked this pull request as ready for review August 10, 2026 09:36
@msonnb
msonnb requested a review from a team as a code owner August 10, 2026 09:36
@msonnb
msonnb requested review from JPeer264 and mydea and removed request for a team August 10, 2026 09:36
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