- ๐บ๐ธUnited States Sunny-Lee
Looks good here, the previous error is no longer happening with this patch. Uploaded 2 videos on various content types and both worked well with a 937.7 MB video.
- ๐บ๐ธUnited States ramu_bharu Bridgewater, Massachusetts
Thanks very much for sharing the patch. I've tested the patch and it works perfectly and I don't get any issues when i upload larger files (100+MB).
- ๐ช๐ธSpain marcoscano Barcelona, Spain
Sounds good to me. Thanks for contributing!
-
marcoscano โ
committed 571be2c4 on 8.x-2.x authored by
m4rcw0lf โ
Issue #3514883 by m4rcw0lf, kevinb623, marcoscano: Assertion fails when...
-
marcoscano โ
committed 571be2c4 on 8.x-2.x authored by
m4rcw0lf โ
- First commit to issue fork.
- ๐บ๐ธUnited States j-barnes
Weโre experiencing this same issue with larger file uploads. When using the widget, files are uploaded directly to S3 client-side. However, the fileInput element still retains the selected file. This means when the form is later submitted via FormData, the file is included again in the request body โ even though it was already uploaded โ which can cause server-side errors such as exceeding post_max_size.
I think the simple solution is just to clear the fileInput value so that FormData doesn't include the file.
- @j-barnes opened merge request.
- First commit to issue fork.
- ๐บ๐ธUnited States kevinb623
I can confirm that this solves the issue of a failed assertion when using s3fs