Skip to content

Sentry module no longer shows user agent, breadcrumbs, or any default integrations #43

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

Closed
robyedlin opened this issue Nov 25, 2018 · 1 comment
Labels

Comments

@robyedlin
Copy link
Contributor

robyedlin commented Nov 25, 2018

Version

2

Steps to reproduce

  1. Create a new nuxt 2 project.
  2. Add @sentry module 2.0.0
  3. Add error
  4. Capture error

What is expected ?

Sentry should

  • Show user agent information
  • Show breadcrumbs

What is actually happening?

Sentry is not

  • Showing user agent information
  • Showing breadcrumbs

Additional comments?

https://github.com/nuxt-community/sentry-module/blob/master/lib/templates/sentry-client.js#L6

This is passing default integrations from the node version: https://github.com/getsentry/sentry-javascript/blob/master/packages/node/src/sdk.ts#L8-L21, but should be pulling from the browser version: https://github.com/getsentry/sentry-javascript/blob/master/packages/browser/src/sdk.ts#L6-L19

This bug report is available on Nuxt community (#c35)
@ghost ghost added the cmty:bug-report label Nov 25, 2018
@robyedlin robyedlin changed the title Sentry module does not show user agent or breadcrumb Sentry module does not show user agent, breadcrumb, or any default integrations Nov 26, 2018
@robyedlin robyedlin changed the title Sentry module does not show user agent, breadcrumb, or any default integrations Sentry module no longer shows user agent, breadcrumb, or any default integrations Nov 26, 2018
@robyedlin robyedlin changed the title Sentry module no longer shows user agent, breadcrumb, or any default integrations Sentry module no longer shows user agent, breadcrumbs, or any default integrations Nov 26, 2018
@robyedlin robyedlin reopened this Nov 26, 2018
@robyedlin
Copy link
Contributor Author

robyedlin commented Nov 26, 2018

@pi0 I know you're busy, would you mind having a quick look at this? I submitted a PR to patch the issue. #44

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

No branches or pull requests

2 participants