NASA-IMPACT / admg-backend

Apache License 2.0
2 stars 0 forks source link

Error trying to access the published view for a certain campaign #602

Closed heidimok closed 7 months ago

heidimok commented 9 months ago

Context

Upon merging the new MI workflow into staging, we ran a Sentry test to catch issues. This is one source of errors found.

Problem

There's an error trying to access the published view for a certain campaign (not sure what's going on there yet). Will need to dig in a bit to explore the problem.

edkeeble commented 8 months ago

https://development-seed-72.sentry.io/issues/4674173153/?environment=dev&project=4504118899179520&query=is%3Aunresolved&referrer=issue-stream&statsPeriod=14d&stream_index=3&utc=true

Sample URL to replicate: https://development-seed-72.sentry.io/issues/?environment=dev&project=4504118899179520&query=is%3Aunresolved%20url%3A%22http%3A%2F%2Fadmgstaging.nasa-impact.net%2Fv2%2Fcampaign%2Fa3be79cd-cb07-4946-8d5b-2fdff101252b%2Fpublished%22&referrer=event-tags&statsPeriod=14d&stream_index=3&utc=true

edkeeble commented 7 months ago

I believe this issue has been resolved from previous changes. Looking back through the historical errors, they seem to be cases where the wrong UUID was being used to refer to a Campaign, but we've made some changes since to address that sort of issue. In any event, I can't replicate it by navigating through the MI. The only way I can get the error again is by going to the bad URL directly.

I will make a change to return a 404 in this case rather than 500 error, since a missing Campaign shouldn't throw a non-descript error.