fix: retire les données potentiellement sensibles de sentry #1845
Conversation
arnaudambro
temporarily deployed
to
review-auto
GitHub Actions
Inactive
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
There was a problem hiding this comment.
Ok c'est bon pour moi, donc simplement : on vire extra presque partout (sauf pour y mettre des IDs)
Est-ce qu'il n'y a pas aussi le problème du contexte ? En effet, on envoie toute l'orga dont encryptedVerificationKey qui peut être flippant et toutes les teams, alors qu'on a ces infos en clair pour nous dans la DB. Donc pas besoin de les leaker. Qu'en penses-tu ?
arnaudambro
force-pushed
the
fix/remove-potentially=sensitive-content-from-sentry
branch
from
22cfdc5
to
512f072
Compare
arnaudambro
temporarily deployed
to
review-auto
GitHub Actions
Inactive
|
|
🎉 Deployment for commit 512f072 : Ingresses
Docker images
|
arnaudambro
deleted the
fix/remove-potentially=sensitive-content-from-sentry
branch
|
🎉 This PR is included in version 1.304.5 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Remarque: tous les
captureque j'ai touché n'ont pas été remontés dans Sentry depuis au moins 90 jours, et certains ne l'ont jamais étéRemarque 2: là normalement, on a plus AUCUNE donnée potentiellement sensible dans sentry - qu'on s'était donné le droit de remonter pour mieux débugger, en période de rodage