zooniverse / theia

Building the next-generation Floating Forests pipeline
3 stars 2 forks source link

Capture usgs connection errors #162

Closed yuenmichelle1 closed 2 years ago

yuenmichelle1 commented 2 years ago

usgs api connection has been finicky in the past, in a prod setting, hard to catch when connection to landsat data api fails, adding a capture message when usgs returns errors

yuenmichelle1 commented 2 years ago

https://sentry.io/organizations/zooniverse-27/issues/3248391985/?project=6185306&query=is%3Aunresolved apologies. you can see message from my local test run.

i think the issue i had with sentry when trying to add the package is that it conflicted with libtiff, so i had to install a different version of a subdependency of sentry and sentry to get it to work.

chelseatroy commented 2 years ago

This looks great. Thank you!