docs(nextjs): added guide for Cloudlfare OpenNext support#15652
docs(nextjs): added guide for Cloudlfare OpenNext support#15652
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
Bundle ReportChanges will increase total bundle size by 53.93kB (0.15%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-server-cjsAssets Changed:
view changes for bundle: sentry-docs-client-array-pushAssets Changed:
|
inventarSarah
left a comment
There was a problem hiding this comment.
Yes, I think it makes sense to set up a separate page for this. Ideally, our quick start guides shouldn't contain any information that can't be found elsewhere, so I think your approach is a good one.
Overall, I think the page looks great 👍. Please add a little more information so that users understand why they need to do this (see my comments).
docs/platforms/javascript/guides/nextjs/best-practices/deploying-on-cloudflare.mdx
Outdated
Show resolved
Hide resolved
docs/platforms/javascript/guides/nextjs/best-practices/deploying-on-cloudflare.mdx
Outdated
Show resolved
Hide resolved
|
Many thanks for the review, I updated the page with your suggestions. |
inventarSarah
left a comment
There was a problem hiding this comment.
thanks for the update -- looks good to me! 🌟
docs/platforms/javascript/guides/nextjs/best-practices/deploying-on-cloudflare.mdx
Outdated
Show resolved
Hide resolved
docs/platforms/javascript/guides/nextjs/best-practices/deploying-on-cloudflare.mdx
Outdated
Show resolved
Hide resolved
…ng-on-cloudflare.mdx Co-authored-by: Sarah Mischinger <sarah@codingwriter.com>
…ng-on-cloudflare.mdx Co-authored-by: Sarah Mischinger <sarah@codingwriter.com>
|
Release |
I won't merge this yet as it partially needs getsentry/sentry-javascript#18336 to be released, but I wanted some initial opinions.
This PR adds new information about supporting Next.js deployments on Cloudflare Workers via Cloudflare's official OpenNext adapter.
During my investigation of Cloudflare Workers not sending any errors or traces for Next.js apps, I found out that it was because Cloudflare workers didn't support
https.requestmodule until recently which would've prevented the SDK from sending anything. Setting the compat date alone fixes most of the sending issues.So I wanted to document this information so users are able to find it more easily.
My findings can be verified by checking out this repo and try setting the
compatdate to anything below August 16th. You don't need a cloudflare account, justwranger devshould give you a similar environment locally to verify that.Question for reviewers: Should this live in special guides like I did here? or maybe add it to the manual setup?