synapse icon indicating copy to clipboard operation
synapse copied to clipboard

Fix content length on federation `/thumbnail` responses

Open H-Shay opened this issue 1 year ago • 1 comments

/_matrix/federation/v1/media/thumbnail/ responses were failing due to the endpoint returning the wrong content-length. This was due to the content-length being calculated off of the length of the original media file, rather than the thumbnail file.

Fixes #17518.

Complement test at https://github.com/matrix-org/complement/pull/732

H-Shay avatar Aug 06 '24 21:08 H-Shay

(Waiting for approval on https://github.com/matrix-org/complement/pull/732 before merging.)

anoadragon453 avatar Aug 12 '24 09:08 anoadragon453

@anoadragon453 https://github.com/matrix-org/complement/pull/732 is approved and merged - can we merge this as well? (sytest failure is a presence test, pretty sure it's not related).

H-Shay avatar Aug 22 '24 18:08 H-Shay

Sorry for the delay here - I've been away. This looks good to merge.

anoadragon453 avatar Aug 28 '24 10:08 anoadragon453