Skip to content

Conversation

chargome
Copy link
Member

@chargome chargome self-assigned this Jun 18, 2025
Copy link

github-actions bot commented Jun 18, 2025

Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against c7d1408

Copy link
Member

@andreiborza andreiborza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed on slack, I'm wondering if we want to actually default the tunnel route option to true. Maybe setting it to true is okay if we mention that they should also switch to an allow list for routes that should be handled by the middleware.

@chargome
Copy link
Member Author

I'd first test this option within our own apps before making it the default within the wizard 👍

Copy link
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR doesn't actually change the default value of the wizard, right? It just changes that if users opt into tunneling, they get the randomized route.
Just asking because we shouldn't default to tunneling due to it potentially heavily increasing traffic on Vercel and therefore cost for users. IIRC the wizard mentions this when it asks users if they want tunneling.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants