Merged in feature/VBV-781-fix-content-disposition-for-files-with-comma (pull request #420)

Fix double content-disposition header for files with commas
This commit is contained in:
Christian Cueni 2024-10-31 09:39:36 +00:00
commit 014ecc31c4
2 changed files with 2 additions and 2 deletions

View File

@ -68,7 +68,7 @@ class DocumentUploadApiTestCase(APITestCase):
self.assertEqual(
response.data["fields"]["Content-Disposition"],
f"attachment; filename={self.test_data['file_name']}",
f'attachment; filename="{self.test_data["file_name"]}"',
)
file_id = response.data["file_id"]

View File

@ -92,7 +92,7 @@ def s3_generate_presigned_post(
Fields={
"acl": acl,
"Content-Type": file_type,
"Content-Disposition": f"attachment; filename={file_name}",
"Content-Disposition": f'attachment; filename="{file_name}"',
},
Conditions=[
{"acl": acl},