-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat(nextjs): Add request data to all edge-capable functionalities #9636
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
Conversation
size-limit report 📦
|
…t-request-data-in-edge
@AbhiPrasad it seems like we can't use the TS built-in |
Is this specifically for Vercel? Just wondering how this would play nicely with other platforms such as Netlify. |
@whitep4nth3r This will currently pretty much only apply to the Next.js SDK but for all hosting platforms. We just named the |
Actually no we didn't because vercel edge has async local storage support, while this is not part of the wintercg spec. So something like cloudflare would not work with the vercel-edge sdk. I think core is our wintercg compatible sdk, |
Right. I keep forgetting this. Sorry! |
This PR adds
RequestData
capabilities to the Vercel Edge SDK. We do not include the capabilities by default until the next major because suddenly including it may leak PII.