-
-
Notifications
You must be signed in to change notification settings - Fork 10
TypeScript error with 7.64.0 version of Sentry packages #190
Comments
Same error here |
Sorry for the delay - I released version 1.1.3 of the profiling package which should fix this error. Please reopen if that is not the case and I'll take a look asap |
I still get this error on v1.2.6. |
Same here, but slightly different error:
|
@lukevers @g-popovic we just released version 1.3.0 which should fix these issues. Please upgrade and let me know if you still see any issues |
@JonasBa I am still having the same issue, but I just realized maybe the problem is me and not this -- I'm using: import Sentry from '@sentry/serverless';
import { ProfilingIntegration } from '@sentry/profiling-node';
Sentry.AWSLambda.init({
... Perhaps I don't actually need to add the profiling integration for the serverless init? I haven't tried it yet, but don't see any reference to adding this one here https://github.com/getsentry/sentry-javascript/tree/master/packages/serverless |
I just upgraded to:
And still got the error:
|
+1 |
Have you ensured that the version of |
Do you mind bumping this to 1.3.2, @sentry/types is now a peer dependency and no longer used by the lib. Lmk if the issue still happens and we can take a look |
install 1.3.2 alone is not sufficiant. Adding @sentry/types seems to do the trick but its not listed in the sentry install instructions for the node sdk |
This is still an issue when following the node docs. I also get:
I'm using init({
dsn: process.env.SENTRY_DSN,
release: version.name,
dist: version.code,
integrations: [
new ProfilingIntegration(),
new Integrations.Http({ tracing: true })
],
...
}); Adding the @sentry/types package then casting got rid of the Typescript error locally: new ProfilingIntegration() as Integration However, when building in CI I get:
|
Weirdly deleting my |
This is still an issue. |
Is there an existing issue for this?
How do you use Sentry?
Sentry Saas (sentry.io)
SDK Version
1.1.2
Link to Sentry event
No response
What environment is your node script running in?
Express + Apollo Server
How is your code deployed and bundled?
Azure App Service
Steps to Reproduce
@sentry
packages,7.64.0
npm install
Sentry.init
Expected Result
The plugin works without type casting
Actual Result
Code snippet:
The text was updated successfully, but these errors were encountered: