getsentry / getsentry/sentry-javascript

Elysia on Node uses Bun's fetch transport

Offen
#24,048 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Bug Bun javascript
Vorherrschende Sprache
TypeScript
Sterne
8.7k
Forks
1.8k
Ø Merge
1 T. 17 Std.
Gemergte PRs (30 T.)
523

Beschreibung

`elysia/src/sdk.ts:51` sets `options.transport = makeFetchTransport` from `@sentry/bun`. On Node that replaces `makeNodeTransport`, losing proxy support (`http_proxy`/`HTTPS_PROXY`), custom CA / `NODE_EXTRA_CA_CERTS` handling, and keep-alive agent tuning. Anyone running Elysia on Node behind a corporate proxy loses all events.

**Work item.** Pick the transport by runtime, the way `getRuntime()` already does. This and E2 both stem from `@sentry/elysia` depending only on `@sentry/bun` while advertising Node support; E1 is unrelated to that and would happen just as readily if Elysia wrapped `@sentry/node`. Worth a short design note on whether the dependency should be conditional.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Start at elysia/src/sdk.ts:51 and trace how getRuntime() selects a runtime. Compare the Node and Bun transport choices, then check the existing Node transport behavior for proxy, CA certificate, and keep-alive support. Done means Elysia uses the appropriate transport on Node without regressing Bun, with the dependency/design implications documented as requested.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
bun, nodejs, typescript
Bereich
backend, networking
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Aktiv
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
70/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.