Skip to content

Commit 0cd1834

Browse files
committed
add changelog entry
1 parent 56825c1 commit 0cd1834

File tree

1 file changed

+22
-0
lines changed

1 file changed

+22
-0
lines changed

CHANGELOG.md

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,28 @@
1010

1111
- "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
1212

13+
### Important Changes
14+
15+
- **fix(node): Avoid double-wrapping http module ([#16177](https://github.com/getsentry/sentry-javascript/pull/16177))**
16+
17+
When running your application in ESM mode, there have been scenarios that resulted in the `http`/`https` emitting duplicate spans for incoming requests. This was apparently caused by us double-wrapping the modules for incoming request isolation.
18+
19+
In order to solve this problem, the modules are no longer monkey patched by us for request isolation. Instead, we register diagnostics*channel hooks to handle request isolation now.
20+
While this is generally not expected to break anything, there is one tiny change that \_may* affect you if you have been relying on very specific functionality:
21+
22+
The `ignoreOutgoingRequests` option of `httpIntegration` receives the `RequestOptions` as second argument. This type is not changed, however due to how the wrapping now works, we no longer pass through the full RequestOptions, but re-construct this partially based on the generated request. For the vast majority of cases, this should be fine, but for the sake of completeness, these are the only fields that may be available there going forward - other fields that _may_ have existed before may no longer be set:
23+
24+
```ts
25+
ignoreOutgoingRequests(url: string, {
26+
method: string;
27+
protocol: string;
28+
host: string;
29+
hostname: string; // same as host
30+
path: string;
31+
headers: OutgoingHttpHeaders;
32+
})
33+
```
34+
1335
## 9.15.0
1436

1537
### Important Changes

0 commit comments

Comments
 (0)