-
Notifications
You must be signed in to change notification settings - Fork 174
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.
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
[document_repository] Uploading file too large not reporting correctly #9158 - fix #9174
[document_repository] Uploading file too large not reporting correctly #9158 - fix #9174
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Shen, thanks for this PR.
I cannot see the "File too large" window appearing.
When trying to send a 5+GB file, I still have this error log:
[Thu Apr 04 10:45:44.134866 2024] [php:warn] [pid 17999] [client ::1:56220] PHP Warning: POST Content-Length of 5214864626 bytes exceeds the limit of 8388608 bytes in Unknown on line 0, referer: http://localhost:8080/document_repository/
There are 2 related |
Still having the same error after applying these config. |
@regis Could you try to upload a 2GB file to test the popup window? It seems like you got the network issue(Chrome) when you upload a 5 GB file. |
@kongtiaowang I will try that. Is the issue on Chrome based browsers? I am mostly using Firefox. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shen and I tested this on 2 VMs.
I had some issues with my config because of the SSH tunnel, it seems.
Using my VM through SSH tunnel raise the error, but using it with the VPN does not.
Brief summary of changes
[document_repository] Uploading file too large not reporting correctly #9158
Testing instructions (if applicable)
upload a large file more than 1 GB, it will popup a error message "file too large"