Skip to content

Commit 7225f42

Browse files
jan-auerkamilogorek
authored andcommitted
fix: Add the DSN protocol to transport options (#1642)
1 parent 06f3ff3 commit 7225f42

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,8 @@
3030
- [node] feat: Make node transactions a pluggable integration with tests
3131
- [node] feat: Transactions handling for RequestHandler in Express/Hapi
3232
- [node] fix: Dont wrap native modules more than once to prevent leaks
33+
- [node] fix: Add the same protocol as dsn to base transport option
3334
- [node] fix: Use getCurrentHub to retrieve correct hub in requestHandler
34-
3535
- [utils] ref: implemented includes, assign and isNaN polyfills
3636

3737
## 4.0.6

packages/node/src/transports/base.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,13 +34,15 @@ export abstract class BaseTransport implements Transport {
3434
...this.options.headers,
3535
};
3636

37+
const dsn = this.api.getDsn();
3738
return {
3839
agent: this.client,
3940
headers,
40-
hostname: this.api.getDsn().host,
41+
hostname: dsn.host,
4142
method: 'POST',
4243
path: this.api.getStoreEndpointPath(),
43-
port: this.api.getDsn().port,
44+
port: dsn.port,
45+
protocol: dsn.protocol,
4446
};
4547
}
4648

0 commit comments

Comments
 (0)