Don't hide exceptions by returning a blank form when something goes wrong -- there's been temporary upload failures for some time, which have been difficult to trace and debug because the error condition has been hidden.

- Legacy-Id: 3514
This commit is contained in:
Henrik Levkowetz 2011-10-25 14:57:35 +00:00
parent 0787ef485a
commit f3d31f0e97

View file

@ -21,13 +21,10 @@ from ietf.utils.mail import send_mail
def submit_index(request):
if request.method == 'POST':
try:
form = UploadForm(request=request, data=request.POST, files=request.FILES)
if form.is_valid():
submit = form.save()
return HttpResponseRedirect(reverse(draft_status, None, kwargs={'submission_id': submit.submission_id, 'submission_hash': submit.get_hash()}))
except IOError:
form = UploadForm(request=request)
form = UploadForm(request=request, data=request.POST, files=request.FILES)
if form.is_valid():
submit = form.save()
return HttpResponseRedirect(reverse(draft_status, None, kwargs={'submission_id': submit.submission_id, 'submission_hash': submit.get_hash()}))
else:
form = UploadForm(request=request)
return render_to_response('submit/submit_index.html',