-
Notifications
You must be signed in to change notification settings - Fork 807
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
[Bug]: Files over 10 MB break when uploading - Missing metadata for chunked upload - affects version Nextcloud client version 3.10.0 #6079
Comments
Same issue here.
Can use web interface to upload the same file without issue. |
Hi @asheroto a debug archive would indeed be highly appreciated, thanks! |
@claucambra Here's mine. |
Hi @claucambra when uploading files larger than 10 MB, they are divided into chunks; when sending a chunk, empty metadata uploadId and uploadPath are sent |
Reading the code this is not the issue. The server checks for these values but the uploadId is created by the server itself and the uploadPath is pulled from the I can confirm that in my own testing with #6084 chunked upload works; once the CI builds the |
I haven't tested with #6084 yet. Even when I disable chunking, the issue still occurs. Whether I do this... sudo -u www-data php occ config:app:set files max_chunk_size --value 2097152000 or this... sudo -u www-data php occ config:app:set files max_chunk_size --value 0 the error message still comes up. I might also mention that I am not using the virtual files but am syncing through "Choose what to sync". Not sure if that matters. debug.zip is the debug archive. Example file is |
Awesome, thanks for your quick response on this! |
Hi @claucambra I tried mirall/3.10.506084 (build 13347) for linux from here nothing changed, have the same issue |
@ffenable could you post both your desktop client's debug archive as well as your server's |
@claucambra I don't see anything interesting there anymore I can say that on version mirall/3.7.50git (build 14472) everything works fine (nothing personal, it was just at hand) |
I am also using Windows FYI |
Hi, @claucambra, i am sorry, but do you have any thoughts on this? can we expect anything in the near future? |
@ffenable not sure if you saw this, but if you downgrade to version 3.9.4 that's a temporary workaround. Sounds like the AppImage file with the fix did not resolve it for you, right? This issue should probably be reopened. |
@asheroto |
Hi, I am also affected by the issue. Why is it closed? |
Downloaded the daily client build from the official NextCloud daily builds URL, still receiving the same problem. NextCloud server 27.1.1 (it's the latest on Docker Hub). I don't have access to a 3.9 client, so I'm uninstalling and reverting to Windows' built in WebDAV client until this is fixed. To use Windows' client as a workaround:
NextCloud will appear in "This PC" in your network locations, next to any mapped Samba drives you may have, like below: You will now be able to upload your large files to NextCloud from the desktop again, using standard file copy operations. |
That's a good alternate workaround to downgrading the client. As far as I can tell, they have not released a new version of the client that includes this fix, but @claucambra may be able to confirm. I would think that the daily build would have included the fix. So if it should, then this bug is not yet resolved. |
Please can someone explain to me how I can patch it myself to publish it to my clients? |
The daily builds are located here: Unfortunately just as @origintopleft has stated, the daily build, even as of today, 2023-10-10, does not fix the issue. I just tried it, version 3.10.50. For now the only workarounds right now: @claucambra merged a commit 3 weeks ago with the fix, so it should have been resolved by now with a daily build, but unfortunately the issue is still present. This issue should be probably be reopened. 😊 |
There are enough reports if this not working that I'm reopening this issue We are considering whether the issue lies at the client end or server end. Will update with more info when we have it |
We have been able to reproduce Thanks all for your patience, and apologies for introducing this bug |
Hey all, we can now provide patched clients containing the aforementioned fix. You may download said client from the links below: Linux: https://cloud.nextcloud.com/s/xndfF9CWPk3jpXW Please let us know if these clients work correctly for you now |
Works for me, thank you very much |
Your patch build provided today for macOS Desktop Client resolves desktop client sync issues for me - we use NextCloud server 26.0.6 and AWS S3 storage. Tested file uploads from 20MB up to 1GB+. All upload fine now using patched client. |
I can confirm from my end that using the patched Windows version does resolve the issue. After updating to the patched version, I saw the same error message again on first sync, but after that it began to work. So maybe it had to clear out the failure first, not sure. I then tried another different file that was 120 MB and it uploaded perfectly fine as well. Much appreciated @claucambra! Not sure if you want to close now or wait until it's been released in the daily builds, but looks like the issue is resolved with this patch. |
We are also impacted by this issue. We would be extremely grateful to see it released without any delay. |
TL;DRIf you are using the desktop client (I only tested it on Windows) and still have the problem after increasing Full messageHello, If like me, you stuble upon this issue with the "Missing metadata for chunked upload" using the desktop client (Windows for my case) and you also tried suggestions from this issue like this one or increasing the php.ini While looking for a solution to my problem, I stumbled upon this documentation page Which led me to this one and more importantly this section This page states that you can create a environment variable named Hope this helps. |
The issue I originally created was resolved in later desktop clients. |
Bug description
Similar to nextcloud/server#38801, but appears to be separate, as it only affects the client and not the server.
Not sure if related to to the PR #3600?There a bug with Nextcloud client version 3.10.0 when uploading files greater than 10 MB. The file gets to 10 MB and then breaks. Nextcloud client logs say...
If the Nextcloud client 3.10.0 is uninstalled and version 3.9.4 is installed then the issue goes away and uploads work perfectly.
Background:
For reference,
.user.ini
is...This workaround was mentioned on nextcloud/server#38801 but doesn't fix this bug. It doesn't matter whether chunked file uploads are set to 0 or any other value, the error still occurs.
Thanks for your time on this. For now the workaround seems to be to downgrade to
3.9.4
.Steps to reproduce
Expected behavior
Syncing should be successful
Which files are affected by this bug
Not sure if this means one of my files or one of the source code files?
Operating system
Windows
Which version of the operating system you are running.
Windows 11
Package
Other
Nextcloud Server version
23.1.0
Nextcloud Desktop Client version
3.10.0
Is this bug present after an update or on a fresh install?
Updated from a minor version (ex. 3.4.2 to 3.4.4)
Are you using the Nextcloud Server Encryption module?
Encryption is Enabled
Are you using an external user-backend?
Nextcloud Server logs
No error messages for this issue appear in the server nextcloud.log file when using the default log level. I can enable debug mode if needed, but this appears to be a client issue not a server issue. I tried downgrading to an older server version but still had the same issue.
Additional info
I can create a debug archive if needed. The issue went away after downgrading the client.
The text was updated successfully, but these errors were encountered: