MirrorHub/synapse
0
0
Fork 1
mirror of https://mau.dev/maunium/synapse.git synced 2024-12-14 22:33:50 +01:00
Code Issues Projects Releases Wiki Activity

Fix zero length media handling (#17570)

Browse source 
Results in:

```
AssertionError: null
  File "synapse/http/server.py", line 332, in _async_render_wrapper
    callback_return = await self._async_render(request)
  File "synapse/http/server.py", line 544, in _async_render
    callback_return = await raw_callback_return
  File "synapse/federation/transport/server/_base.py", line 369, in new_func
    response = await func(
  File "synapse/federation/transport/server/federation.py", line 826, in on_GET
    await self.media_repo.get_local_media(
  File "synapse/media/media_repository.py", line 473, in get_local_media
    await respond_with_multipart_responder(
  File "synapse/media/_base.py", line 353, in respond_with_multipart_responder
    assert content_length is not None
```
This commit is contained in:
Erik Johnston 2024-08-19 15:06:44 +01:00 committed by GitHub
parent a5d25bb623
commit 993644ded0
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
changelog.d/17570.bugfix Normal file
View file

@ -0,0 +1 @@
Fix bug where we would respond with an error when a remote server asked for media that had a length of 0, using the new multipart federation media endpoint.

2
synapse/media/media_storage.py
View file

@ -544,7 +544,7 @@ class MultipartFileConsumer:
Calculate the content length of the multipart response Calculate the content length of the multipart response
in bytes. in bytes.
""" """
if not self.length: if self.length is None:
return None return None
# calculate length of json field and content-type, disposition headers # calculate length of json field and content-type, disposition headers
json_field = json.dumps(self.json_field) json_field = json.dumps(self.json_field)