-
Notifications
You must be signed in to change notification settings - Fork 29
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
chunked upload fails on with oCIS 6 #643
Comments
Just for curiosity: What are larger files? 500MB, 1GB? |
In this case it was 56,3 MiB. Actually I can't recap the actual size where Web switches from single POST upload to chunked TUS upload. |
@micbar Is that some information you or your team can provide? |
The default is documented in the docs. I think it is 10MB |
Yes it is. Funnily this is no problem on https://ocis.ocis.rolling.owncloud.works: In Minikube, I get:
For whatever reason I see on the continuous deployment:
|
Actually I need to test in Minikube with a file that I didn't upload before, than I also can see a POST before the patch: The log for this case (but different file) looks like this:
|
The nginx logs while uploading a file for the first time:
|
Also worth noting is the fancy animation which jumps backwards: Screen.Recording.2024-08-08.at.18.52.56.mov |
Found it. It was cors. Additionally, I set
in my values.yaml. |
Just for curiosity: Why hasn't this been an issue with oCIS 5? I meant we also did TUS uploads in oCIS 5, didn't we? |
Good question. We bumped the tusd package recently to newer version. Also we did some improvements to cors lately. My best guess that it wasn't working as it should before. Now that we've updated the dependencies the feature works properly and therefore throws an error. (Just a guess though...) |
Closing as #675 got merged. Please reopen if issue persists. |
This are the logs when trying to continue the upload after the first time abort:
|
Here a short script to create test files:
|
I couldn't manage to upload larger files because they all end up with 502 Bad Gateway on the PATCH request that follows the first POST request when using chunked upload with TUS
cc @case0sh @dschmidt
The text was updated successfully, but these errors were encountered: