Skip to content
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

chore: add log on coral stream raising an error #1293

Closed

Conversation

juanSanchezAlcala
Copy link
Contributor

@juanSanchezAlcala juanSanchezAlcala commented Oct 4, 2023

If the initialisation of the client object raise and exception we are not caching it and not logging it. With these changes we track the error in that case.

Why ?
We do this because a lot of users are reporting coral initialisation errors , we need to collect data to support this reports

@juanSanchezAlcala juanSanchezAlcala force-pushed the CI-1837-investigate-coral-issues-in-the-apps branch from b8125d7 to 08404d9 Compare October 4, 2023 15:05
@notlee notlee temporarily deployed to origami-webs-ci-1837-in-5jbhxe October 4, 2023 15:06 Inactive
@juanSanchezAlcala juanSanchezAlcala force-pushed the CI-1837-investigate-coral-issues-in-the-apps branch 2 times, most recently from 5663f77 to 773cbc5 Compare October 4, 2023 15:07
@notlee
Copy link
Contributor

notlee commented Oct 4, 2023

Hey @juanSanchezAlcala, you may also be interested in o-errors to implement this. Sends errors to Sentry. It's an old library and I have no idea if/how it's used by Customer Products 🙈 So, just a suggestion to take a quick look. I have no problem with a custom tracking event

@notlee notlee temporarily deployed to origami-webs-ci-1837-in-5jbhxe October 5, 2023 07:13 Inactive
@juanSanchezAlcala juanSanchezAlcala force-pushed the CI-1837-investigate-coral-issues-in-the-apps branch from 1b906b8 to 4fc79f4 Compare October 5, 2023 07:58
@notlee notlee temporarily deployed to origami-webs-ci-1837-in-5jbhxe October 5, 2023 07:58 Inactive
@juanSanchezAlcala juanSanchezAlcala force-pushed the CI-1837-investigate-coral-issues-in-the-apps branch from 4fc79f4 to 2744355 Compare October 5, 2023 07:59
@notlee notlee temporarily deployed to origami-webs-ci-1837-in-5jbhxe October 5, 2023 07:59 Inactive
@notlee
Copy link
Contributor

notlee commented Aug 21, 2024

Closing this as it's been open for a while. Please let me know if you would like to revisit this PR.

@notlee notlee closed this Aug 21, 2024
@notlee notlee deleted the CI-1837-investigate-coral-issues-in-the-apps branch December 12, 2024 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants