You need to sign in or sign up before continuing.
Fix Content-Length for mail attachment and passport download
What does this MR do?
It fixes the wrong Content-Length header for mailbox attachments and passports, what lead to failing download
How confident are you it won't break things if deployed?
Sure
Links to related issues
https://yunity.slack.com/archives/CDFSR3Y4U/p1681323486175869
How to test
Already tested on beta and production
Checklist
-
added a test, or explain why one is not needed/possible... -
no unrelated changes -
asked someone for a code review -
set a "for:" label to indicate who will be affected by this change -
added to the next milestone (see https://gitlab.com/foodsharing-dev/foodsharing/-/milestones, unless it has a "for:Dev" label) -
added an entry to CHANGELOG.md
Edited by Stefan C
Merge request reports
Activity
changed milestone to %Release Jackfrucht
(September 2023)added forFS Community label
assigned to @stcz
added 1 commit
- ed7fc7bc - Add wrong header in PassportGeneratorControl.php
added 6 commits
-
e9829b3a...edac9dab - 2 commits from branch
master
- 4a715d27 - Fix Mailbox attachment download
- cd51cfe4 - Add CHANGELOG.md
- fecf428c - Add wrong header in PassportGeneratorControl.php
- e2b5923b - Update CHANGELOG.md
Toggle commit list-
e9829b3a...edac9dab - 2 commits from branch
enabled an automatic merge when the pipeline for e2b5923b succeeds
Please register or sign in to reply