getsentry / getsentry/sentry-javascript

Nuxt dev server loads DB drivers before Sentry.init()

未关闭
#23,876 2 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
Bug javascript Node.js
主要语言
TypeScript
星标
8.7k
派生
1.8k
平均合并
1 天 17 小时
30 天内合并 PR
523

描述

In `nuxt dev`, `ioredis` and `mysql` never get DB spans. The SDK starts fine and `http.server` transactions arrive, but the DB spans are missing.

It's a load-order problem. Nitro generates `.nuxt/dev/sentry.server.config.mjs`, the file `nuxt dev` passes to `--import`, as a list of imports for the whole dev server:

```js
import './index.mjs'; // the whole Nitro dev server
// … a few more imports …
import '…/node_modules/ioredis/built/index.js';
import '…/node_modules/mysql/index.js';
// … more imports, then Sentry.init() somewhere inside
```

ESM runs all imports before any code, so both drivers are loaded by the time `Sentry.init()` runs, and the runtime hook can't instrument a module that loaded first. A probe on `Module.prototype._compile` confirms it: when `ioredis` compiles, `globalThis.__SENTRY_ORCHESTRION__` doesn't exist yet.

This isn't only about orchestrion. Anything that needs `Sentry.init()` to run before a dependency loads is unreliable in `nuxt dev`.

**Repro:** in `dev-packages/e2e-tests/test-applications/nuxt-4`, run `bash ./nuxt-start-dev-server.bash && TEST_ENV=development playwright test db-drivers`. Both tests time out; the dev pass is filtered to `environment` today for this reason.

Found while working on JS-3233 (#22857). Blocks the nuxt-4 dev DB tests (#23836).

贡献指南

打开贡献指南

调研方向

Reproduce the failure from dev-packages/e2e-tests/test-applications/nuxt-4 with nuxt-start-dev-server.bash and the db-drivers Playwright test. Inspect the generated .nuxt/dev/sentry.server.config.mjs and the Nuxt dev entry point to trace import timing; done means the DB-driver tests pass in development without the environment filter.

由索引模型根据 Issue 内容生成。

评估

技术栈
mysql, nodejs, nuxt, playwright, typescript
领域
backend, observability-sre, testing-qa
Issue 类型
缺陷
难度
4/5
预计耗时
3-5 天
活跃度
活跃
描述清晰度
基本清楚
新手友好度
55/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。