New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix attachments.test.js #89

Merged

DanSchlachter merged 1 commit into main from fix-attachments-test

Feb 28, 2025

Contributor

hm23 commented Feb 27, 2025 •

edited

Loading

Test suite failed because an attachment was uploaded without Content-Length header. Missing Content-Length header leads to a 403 (Invalid Content Size) in the attachments plugin since the following change: https://github.com/cap-js/attachments/pull/105/files#diff-11af61311ab29f43ed180a25e84cbae6a92c72a244b8c4f717d573ec930ff5acR90-R103


          Fix attachments.test.js

e87896f

hm23 requested a review from DanSchlachter

February 27, 2025 16:13

DanSchlachter approved these changes

View reviewed changes

DanSchlachter merged commit d1bd38a into main

4 checks passed

DanSchlachter deleted the fix-attachments-test branch

February 28, 2025 08:31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet