Skip to content

Commit

Permalink
CONCD-712 previous attempt was not catching the exception, I'm going …
Browse files Browse the repository at this point in the history
…to just catch all here (#2303)
  • Loading branch information
rasarkar authored Mar 1, 2024
1 parent cca11c6 commit 65a6bd8
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions exporter/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,10 @@

import bagit
import boto3
from asgiref.sync import sync_to_async
from django.conf import settings
from django.contrib.admin.views.decorators import staff_member_required
from django.contrib.postgres.aggregates.general import StringAgg
from django.core.exceptions import SynchronousOnlyOperation
from django.db.models import OuterRef, Subquery
from django.http import HttpResponse, HttpResponseRedirect
from django.utils.decorators import method_decorator
Expand Down Expand Up @@ -234,8 +234,12 @@ def get(self, request, *args, **kwargs):
return export_to_csv_response(
"%s.csv" % self.kwargs["campaign_slug"], headers, data
)
except SynchronousOnlyOperation as e:
logger.info("Failed to export csv, error was: %s", e)
except Exception:
logger.info("Attemping to convert function to async")
export_csv_async = sync_to_async(export_to_csv_response)
return export_csv_async(
"%s.csv" % self.kwargs["campaign_slug"], headers, data
)


class ExportItemToBagIt(TemplateView):
Expand Down

0 comments on commit 65a6bd8

Please sign in to comment.