darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-24 08:16:51 -04:00
Code Issues Packages Projects Releases Wiki Activity

Give Flask bytes to send, not string

Browse Source 
For some reason this passed the tests, but didn’t work running locally.
This commit is contained in:
Chris Hill-Scott
2020-03-16 13:47:10 +00:00
parent f563c4a181
commit 00f953de6d
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/main/views/uploads.py
View File

@@ -3,7 +3,7 @@ import itertools
import json import json
import urllib import urllib
import uuid import uuid
from io import BytesIO, StringIO from io import BytesIO
from zipfile import BadZipFile from zipfile import BadZipFile
from flask import ( from flask import (
@@ -449,7 +449,7 @@ def contact_list(service_id, contact_list_id):
def download_contact_list(service_id, contact_list_id): def download_contact_list(service_id, contact_list_id):
contact_list = ContactList.from_id(contact_list_id, service_id=service_id) contact_list = ContactList.from_id(contact_list_id, service_id=service_id)
return send_file( return send_file(
filename_or_fp=StringIO(contact_list.contents), filename_or_fp=BytesIO(contact_list.contents.encode('utf-8')),
attachment_filename=contact_list.saved_file_name, attachment_filename=contact_list.saved_file_name,
as_attachment=True, as_attachment=True,
) )