vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: update Content-Disposition header for binary as attachment

Browse source
This commit is contained in:
ljain112 2026-01-21 19:10:12 +05:30 • committed by Sagar Vora
parent a73d3f54cf
commit 4a9af04f94
2 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
frappe/tests/test_api.py
View file

@ -483,7 +483,9 @@ class TestAPIResponse(FrappeAPITestCase):
self.assertEqual(response.status_code, 200)
self.assertEqual(response.headers["content-type"], "application/octet-stream")
self.assertGreater(cint(response.headers["content-length"]), 0)
self.assertEqual(response.headers["content-disposition"], f'filename="{encoded_filename}"')
self.assertEqual(
response.headers["content-disposition"], f'attachment; filename="{encoded_filename}"'
)
def test_download_private_file_with_unique_url(self):
test_content = frappe.generate_hash()

2
frappe/utils/response.py
View file

@ -169,7 +169,7 @@ def as_binary():
response.mimetype = "application/octet-stream"
filename = frappe.response["filename"]
filename = filename.encode("utf-8").decode("unicode-escape", "ignore")
response.headers.add("Content-Disposition", None, filename=filename)
response.headers.add("Content-Disposition", "attachment", filename=filename)
response.data = frappe.response["filecontent"]
return response