-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
feat(vercel-edge): Add dedupe as default integration #12957
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just out of personal curiosity: Dedupe is only enabled by default in browser and deno, not in node. Should we change this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't want to block this in the meantime. Sounds like a good idea to add it to vercel-edge from my PoV
I didn't realize it was not enabled in Node. Let me write up a GH issue to discuss this with the wider group. I think it's fine to release this for now because constraining volume of events sent from serverless env is very valuable. |
Add the dedupe integration as a default integration to
vercel-edge
.Saw this discrepancy while working on the cloudflare sdk.