-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Group similar error reports #5736
Comments
the same happens for different extension ids, as example:
here we can replace extension ids with |
Hi @ioanmo226, can you please work on this one before #5749, so it can be included in the 8.5.6 release to better distinguish client error reports. Thanks! |
Ok, got it |
sosnovsky
pushed a commit
that referenced
this issue
May 31, 2024
* feat: group similar error reports * fix: remove unnecessary line * fix: simplify code * fix: pr reviews
ioanmo226
added a commit
that referenced
this issue
Jun 3, 2024
* feat: group similar error reports * fix: remove unnecessary line * fix: simplify code * fix: pr reviews
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.The text was updated successfully, but these errors were encountered: