Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

captureConsoleIntegration does not works in v4.18.0 #833

Closed
3 tasks done
panther7 opened this issue Feb 21, 2024 · 4 comments · Fixed by #835
Closed
3 tasks done

captureConsoleIntegration does not works in v4.18.0 #833

panther7 opened this issue Feb 21, 2024 · 4 comments · Fixed by #835

Comments

@panther7
Copy link

panther7 commented Feb 21, 2024

Is there an existing issue for this?

How do you use Sentry?

Self-hosted/on-premise

Electron SDK Version

4.18.0

Electron Version

28.2.3

What platform are you using?

Windows

Link to Sentry event

No response

Steps to Reproduce

Works properly in @sentry/[email protected]

Sentry config (in main process):

import * as Sentry from '@sentry/electron/main'; // v4.18.0
import { captureConsoleIntegration } from '@sentry/integrations'; // v7.102.0

Sentry.init({
    // ...
    ipcMode: Sentry.IPCMode.Protocol,
    integrations: [
        captureConsoleIntegration({ levels: ['error'] }),
        new Sentry.Integrations.MainProcessSession({ sendOnCreate: true }),
    ],
    // ...
}

Expected Result

console.error('test'); // capture error

Actual Result

console errors are not captured

@lforst
Copy link
Member

lforst commented Feb 23, 2024

Hm, sounds weird, can you try using the 7.101.0 version of the integrations package? That one aligns with the one the electron SDK is using. Thanks!

@panther7
Copy link
Author

panther7 commented Feb 23, 2024

@lforst done. Tested on Electron v28.2.3.

Sentry / Integrations 7.101.0 7.102.0 7.102.1
4.17.0
4.18.0

@lforst
Copy link
Member

lforst commented Feb 26, 2024

@panther7 Thanks for testing! I recommend reverting the integrations package back to 7.101.0. The electron package has a certain compatibility range with the other Sentry SDK packages and that one happens to only include 7.101.0 right now.

@timfish
Copy link
Collaborator

timfish commented Feb 26, 2024

Once v4.19.0 of the Electron SDK is released, you'll be able to use v7.102.1 of @sentry/integrations

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants