FlowCrypt / flowcrypt-browser

FlowCrypt Browser extension for Chrome and Firefox
https://flowcrypt.com
Other
372 stars 48 forks source link

Group similar error reports #5736

Closed sosnovsky closed 1 month ago

sosnovsky commented 1 month ago

There are many error reports which are sent in doSendErrorToSharedTenantFes with similar message like 429 when GET-ing https://www.googleapis.com/gmail/v1/users/me/threads/18fa6109b0f0d121 (no body), which has the only difference - thread id.

For such cases we should just replace such parameters with THREAD_ID, so similar reports will be grouped on our backend.

sosnovsky commented 1 month ago

the same happens for different extension ids, as example:

here we can replace extension ids with EXTENSION_ID

sosnovsky commented 1 month ago

Hi @ioanmo226, can you please work on this one before https://github.com/FlowCrypt/flowcrypt-browser/pull/5749, so it can be included in the 8.5.6 release to better distinguish client error reports. Thanks!

ioanmo226 commented 1 month ago

Ok, got it

ioanmo226 commented 1 month ago

Let me confirm Do I have to replace those EXTENSION_ID and THREAD_ID in url field or trace field? Or both? image

sosnovsky commented 1 month ago

They should be changed in url and message fields, here is screenshot from error reports dashboard:

Screenshot 2024-05-31 at 11 45 00

Currently I noticed 3 paths which should be replaced (maybe in the future there will be another ones):