getsentry / getsentry/sentry-javascript
Auto instrumentation in Cloudflare with Vite plugin
- Lenguaje dominante
- TypeScript
- Estrellas
- 8.7k
- Forks
- 1.8k
- Merge medio
- 1 d 17 h
- PR fusionados (30 d)
- 515
Descripción
### Problem Statement
As of now people need to manually wrap their worker with `withSentry` or `instrumentDurableObjectWithSentry` in their source code, which needs intervention. Also the config needs to be adapted in multiple places, and users already have helper functions that reuses their config.
### Solution Brainstorm
Vite could auto wrap all functions, as during build time we have access to `wrangler.jsonc` and could therefore read out which classes are meant to be wrapped with which function of ours. That would mean that users only have to add our Cloudflare Vite plugin, add a `instrument.server.ts` or similar, which could include the following code:
```js
import { defineCloudflareOptions } from '@sentry/cloudflare';
export default defineCloudflareOptions((env) => ({
dsn: env.SENTRY_DSN,
tracesSampleRate: 1,
}));
```
Since the config is a callback, which makes it possible to use the environment variables `defineCloudflareOptions` could be the solution to use these variables. On top if users wish to have different configs per worker, they could use it differently
```js
import { defineCloudflareOptions } from '@sentry/cloudflare';
// default worker
export default defineCloudflareOptions((env) => ({
dsn: env.SENTRY_DSN,
tracesSampleRate: 1,
}));
// matched name as the binding from the default worker entrypoint, will warn if it doesn't exist
export const MY_WORKER = defineCloudflareOptions((env) => ({
dsn: env.SENTRY_DSN,
tracesSampleRate: 1,
}));
```
### Additional Context
_No response_
### Priority
React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding `+1` or `me too`, to help us triage it.
Guía de contribución
Evaluación
Este issue todavía no se ha evaluado.