cozy-labs / cozy-desktop

File Synchronisation for Cozy on Desktop and Laptop
https://cozy-labs.github.io/cozy-desktop/
GNU Affero General Public License v3.0
99 stars 45 forks source link

fix: Setup Sentry only when Desktop is onboarded #2314

Open taratatach opened 1 year ago

taratatach commented 1 year ago

We need a valid Desktop config to setup our Sentry integration as we use the Cozy URL to determine the domain, the instance and the environment of the Cozy to tag our events.

Since we want to receive events as soon as possible, we setup Sentry very early and, when Desktop is not onboarded already, we get an error since we don't have the required data. This is not a big deal but the error is printed in the console.

We'll now make sure we have a valid configuration before trying to setup the integration and we'll also make sure to set it up after the onboarding is done to receive events without the need for an app restart.

Please make sure the following boxes are checked: