Handle content types with parameters. (#16440)

This commit is contained in:
Patrick Cloke 2023-10-11 07:50:34 -04:00 committed by GitHub
parent f1e43018b7
commit d6b7d49a61
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 22 additions and 2 deletions

1
changelog.d/16440.bugfix Normal file
View File

@ -0,0 +1 @@
Properly return inline media when content types have parameters.

View File

@ -148,7 +148,9 @@ def add_file_headers(
# A strict subset of content types is allowed to be inlined so that they may # A strict subset of content types is allowed to be inlined so that they may
# be viewed directly in a browser. Other file types are forced to be downloads. # be viewed directly in a browser. Other file types are forced to be downloads.
if media_type.lower() in INLINE_CONTENT_TYPES: #
# Only the type & subtype are important, parameters can be ignored.
if media_type.lower().split(";", 1)[0] in INLINE_CONTENT_TYPES:
disposition = "inline" disposition = "inline"
else: else:
disposition = "attachment" disposition = "attachment"

View File

@ -42,18 +42,35 @@ class GetFileNameFromHeadersTests(unittest.TestCase):
class AddFileHeadersTests(unittest.TestCase): class AddFileHeadersTests(unittest.TestCase):
TEST_CASES = { TEST_CASES = {
# Safe values use inline.
"text/plain": b"inline; filename=file.name", "text/plain": b"inline; filename=file.name",
"text/csv": b"inline; filename=file.name", "text/csv": b"inline; filename=file.name",
"image/png": b"inline; filename=file.name", "image/png": b"inline; filename=file.name",
# Unlisted values are set to attachment.
"text/html": b"attachment; filename=file.name", "text/html": b"attachment; filename=file.name",
"any/thing": b"attachment; filename=file.name", "any/thing": b"attachment; filename=file.name",
# Parameters get ignored.
"text/plain; charset=utf-8": b"inline; filename=file.name",
"text/markdown; charset=utf-8; variant=CommonMark": b"attachment; filename=file.name",
# Parsed as lowercase.
"Text/Plain": b"inline; filename=file.name",
# Bad values don't choke.
"": b"attachment; filename=file.name",
";": b"attachment; filename=file.name",
} }
def test_content_disposition(self) -> None: def test_content_disposition(self) -> None:
for media_type, expected in self.TEST_CASES.items(): for media_type, expected in self.TEST_CASES.items():
request = Mock() request = Mock()
add_file_headers(request, media_type, 0, "file.name") add_file_headers(request, media_type, 0, "file.name")
request.setHeader.assert_any_call(b"Content-Disposition", expected) # There should be a single call to set Content-Disposition.
for call in request.setHeader.call_args_list:
args, _ = call
if args[0] == b"Content-Disposition":
break
else:
self.fail(f"No Content-Disposition header found for {media_type}")
self.assertEqual(args[1], expected, media_type)
def test_no_filename(self) -> None: def test_no_filename(self) -> None:
request = Mock() request = Mock()