Add per request global context #1789
Conversation
Co-authored-by: Tobbe Lundberg <tobbe@tlundberg.com>
|
Test summary
Run details
View run in Cypress Dashboard ➡️ This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
| | AsyncLocalStorage<Map<string, GlobalContext>> = undefined | ||
|
|
||
| export const usePerRequestContext = () => | ||
| process.env.SAFE_GLOBAL_CONTEXT !== '1' |
There was a problem hiding this comment.
Should we set this to '1' in the netlify and vercel deploy setup scripts?
There was a problem hiding this comment.
Yeah - because there is a performance impact, but I'm not 100% certain that I want to trust Netlify and Vercel. So I would like to make this the default everywhere, and give people the ability to opt-out with the understand that it could be a problem.
@peterp @Tobbe we definitely need some documentation about all this. Maybe we expand this doc to be API & Serverless Functions... https://redwoodjs.com/docs/serverless-functions |
This adds the experimental NodsJS async local storage API so that we can have a per-request global context.