nuxt-community / sentry-module

Sentry module for Nuxt 2
https://sentry.nuxtjs.org
MIT License
501 stars 114 forks source link

fix(tracing): initialize server side tracing correctly #517

Closed rchl closed 1 year ago

rchl commented 1 year ago

The @sentry/tracing package has to be imported so that some tracing extensions are added. Also forgot to pass tracing option on the server so tracing was never enabled there.

Fixes #508

codecov[bot] commented 1 year ago

Codecov Report

Merging #517 (f51d73b) into main (5472233) will decrease coverage by 1.70%. The diff coverage is n/a.

:exclamation: Current head f51d73b differs from pull request most recent head b948aed. Consider uploading reports for the commit b948aed to get more accurate results

@@            Coverage Diff             @@
##             main     #517      +/-   ##
==========================================
- Coverage   72.88%   71.18%   -1.70%     
==========================================
  Files           1        1              
  Lines          59       59              
  Branches       24       24              
==========================================
- Hits           43       42       -1     
- Misses         13       14       +1     
  Partials        3        3              
Impacted Files Coverage Δ
lib/module.js 71.18% <0.00%> (-1.70%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.