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

More resilience with "Failed host lookup: 'static.openfoodfacts.org'" (svg icons) #6020

Open
sentry-io bot opened this issue Dec 7, 2024 · 4 comments

Comments

@sentry-io
Copy link

sentry-io bot commented Dec 7, 2024

Sentry Issue: SMOOTHIE-46M

Sometimes, for some reason we don't manage to retrieve svg icons from the server.
Not a big deal for Smoothie, as we have a local fallback icon.
The problem is that we get flooded by related sentry errors (all icons failing in the same session), that are relatively harmless but may hide more important but less common issues - just by being the top sentry errors.
A solution would be to sent only one "Failed host lookup: static..." by session. We'll still get the annoying message that the server wasn't reachable, but with more parcimony.

OSError: OS Error: No address associated with hostname, errno = 7
ClientException: ClientException with SocketException: Failed host lookup: 'static.openfoodfacts.org' (OS Error: No address associated with hostname, errno = 7), uri=https://static.openfoodfacts.org/images/icons/dist/car.svg
  File "stacktrace_utils.dart", line 10, in getCurrentStackTrace
  File "sentry_exception_factory.dart", line 44, in SentryExceptionFactory.getSentryException
  File "sentry_client.dart", line 235, in SentryClient._prepareEvent
  File "sentry_client.dart", line 119, in SentryClient.captureEvent
  File "hub.dart", line 173, in Hub.captureException
@monsieurtanuki monsieurtanuki changed the title More resilience with "Failed host lookup: 'static.openfoodfacts.org'" More resilience with "Failed host lookup: 'static.openfoodfacts.org'" (svg icons) Dec 7, 2024
@monsieurtanuki
Copy link
Contributor

Top 1 sentry issue this week.

@monsieurtanuki
Copy link
Contributor

Supposed to be closed since... March 4th.
Too bad we only released it 9 months later in 4.17.0.

@github-project-automation github-project-automation bot moved this from 💬 To discuss and validate to 🎊 Done in 🤳🥫 The Open Food Facts mobile app (Android & iOS) Dec 14, 2024
@monsieurtanuki
Copy link
Contributor

Top 1 sentry issue this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

1 participant