sentry-javascript
sentry-javascript copied to clipboard
feat(node): Add `tedious` integration
Resolves #13316
Implement Tedious OTL instrumentation in packages/node.
- [x] If you've added code that should be tested, please add tests.
- [x] Ensure your code lints and the test suite passes (
yarn lint) & (yarn test).
The check Node (22) (TS 3.8) Integration Tests is a consistent fail - observed in previous commits as well.
This is b/c:
tedious 15.1.3is the newest version that supportsnode >= 14.- That version of tedious depends on
@azure/identity 2.1.0. - Which in turn depends on
@azure/msal-node 1.18.4. This (deprecated) package doesn't support node 22, causing the failed check.
We could bump tedious to 16.x.x or later, but those versions lacks support for node 14 and/or 16.
We could bump
tediousto 16.x.x or later, but those versions lacks support for node 14 and/or 16.
I recommend we add support for a newer version that may not be supporting as many older node versions!
I recommend we add support for a newer version that may not be supporting as many older node versions!
Noted! I'll look into it then.
I recommend we add support for a newer version that may not be supporting as many older node versions!
Noted! I'll look into it then.
Bumped to 18.6.1, which supports node >=18.
Hey @Zen-cronic, apologies for the delay in reviewing this! I'd like to help push this forward so I'm going to push some commits to this branch to resolve conflicts and get it up to date.
awesome!