Skip to content

[Feature Request] Add Console integration to @sentry/cloudflare #15439

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Fwang36 opened this issue Feb 18, 2025 · 6 comments · Fixed by #16024
Closed

[Feature Request] Add Console integration to @sentry/cloudflare #15439

Fwang36 opened this issue Feb 18, 2025 · 6 comments · Fixed by #16024

Comments

@Fwang36
Copy link
Member

Fwang36 commented Feb 18, 2025

Problem Statement

Filing this request on behalf of a user.

The Console integration, which is listed as default-on is missing from the Cloudflare SDK. Can we get it added? Users currently have to manually add console logs to breadcrumbs.

Solution Brainstorm

Add the Console integration to @sentry/cloudflare

@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Feb 18, 2025
@Lms24
Copy link
Member

Lms24 commented Feb 18, 2025

Seems like a reasonable request on first glance! I'll tag @AbhiPrasad - are you aware of console limitations in the cloudflare SDK (or another good reason not to add this integration)?

@Fwang36
Copy link
Member Author

Fwang36 commented Feb 19, 2025

Small update from the user. They were able to install the node sdk and import the Console integration and use it successfully in the cloudflare sdk. They say that to get it to work, it requires Cloudflare's "nodejs_compat" to be set to at least "2024-09-23" and a recent version of wrangler.

@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Feb 19, 2025
@Lms24
Copy link
Member

Lms24 commented Feb 20, 2025

Good to know, thanks!

@AbhiPrasad
Copy link
Member

are you aware of console limitations in the cloudflare SDK (or another good reason not to add this integration)?

Nope, we should be able to add this, just copy our implementation from the other SDKs.

this is a good reason to unify all of this as well.

@connorads
Copy link

FYI these docs will need updating when making this change
https://docs.sentry.io/platforms/javascript/guides/cloudflare/configuration/integrations/console/

This integration only works inside server environments (Node.js, Bun, Deno).
Image

Copy link
Contributor

A PR closing this issue has just been released 🚀

This issue was referenced by PR #16024, which was included in the 9.13.0 release.

@stephanie-anderson stephanie-anderson removed the Package: cloudflare Issues related to the Sentry Cloudflare Workers SDK label Apr 16, 2025
connorads added a commit to connorads/sentry-docs that referenced this issue Apr 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment