io-app icon indicating copy to clipboard operation
io-app copied to clipboard

feat: [IOPLT-305] Add Sentry Tool for Crash reporting

Open CrisTofani opened this issue 9 months ago β€’ 2 comments

Short description

This PR adds the Sentry SDK to track and handle crash informations.

List of changes proposed in this pull request

  • Feature A
  • Feature B

How to test

Describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

CrisTofani avatar May 09 '24 13:05 CrisTofani

Affected stories

  • 🌟 IOPLT-305: Come DEV dell'APP IO voglio usufruire dei servizi di Sentry al fine di poter inviare eventi utili alla redazione di crash reporting subtask of

Generated by :no_entry_sign: dangerJS against 5cd0cf318fcb37faa7cd605ab2e458b78d2be1d3

pagopa-github-bot avatar May 09 '24 13:05 pagopa-github-bot

Codecov Report

Attention: Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 48.67%. Comparing base (4f204b4) to head (292a605). Report is 351 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5756      +/-   ##
==========================================
+ Coverage   48.42%   48.67%   +0.24%     
==========================================
  Files        1488     1760     +272     
  Lines       31617    35568    +3951     
  Branches     7669     8615     +946     
==========================================
+ Hits        15311    17311    +2000     
- Misses      16238    18187    +1949     
- Partials       68       70       +2     
Files Coverage Ξ”
ts/config.ts 100.00% <100.00%> (ΓΈ)
ts/screens/profile/DeveloperModeSection.tsx 25.35% <0.00%> (+20.87%) :arrow_up:

... and 1119 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more Ξ” = absolute <relative> (impact), ΓΈ = not affected, ? = missing data Powered by Codecov. Last update 7fb5d53...292a605. Read the comment docs.

codecov[bot] avatar May 09 '24 13:05 codecov[bot]

:tada: All dependencies have been resolved !

dpulls[bot] avatar Jul 23 '24 07:07 dpulls[bot]

We found that in "breadcrumbs" we log the installationID ❌ @shadowsheep1 I added a new scrubbing rule on sentry to remove the id from API path, so we can be safe for future events!

Thanks for the catch! πŸ˜„

CrisTofani avatar Jul 23 '24 09:07 CrisTofani