Skip to content
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

Stable9.1 chunked web upload #26970

Closed
wants to merge 11 commits into from
Closed

Conversation

butonic
Copy link
Member

@butonic butonic commented Jan 18, 2017

backport of #26306 to stable9.1

requires a few more commits to first enable PUT uploads.

manually tested in IE11, chrome canary and firefox.

cc @michaelstingl @PVince81

Vincent Petry added 11 commits January 18, 2017 17:12
- uses PUT method with jquery.fileupload for regular and public file
  lists
- for IE and browsers that don't support it, use POST with iframe
  transport
- implemented Sabre plugin to handle iframe transport and redirect the
  embedded PUT request to the proper handler
- added RFC5995 POST to file collection with "add-member" property to
  make it possible to auto-rename conflicting file names
- remove obsolete ajax/upload.php and obsolete ajax routes
Not needed any more in IE >= 11
Hacked around Blueimp's jquery.fileupload to make it work with our new
chunking API.
…26069)

The web UI now uses for PUT uploads which aren't restricted by PHP's
upload_max_filesize and post_max_size
Properly trigger the "stop" even from the uploader.
Also update storage stats at the end of all uploads instead of for each
upload.
Removes the need for POST to collection which would hit against upload
limits.

The client tries to auto rename the file by adding a suffix "(2)".
It tries to use the file list on the client side to guess a
suitable name. In case a file still cannot be uploaded and creates a
conflict, which can happen when the file was concurrently uploaded, the
logic will continue increasing the suffix.
@butonic
Copy link
Member Author

butonic commented Jan 18, 2017

✔️ with 4.6GB on IE11 🎉 @PVince81 is awesome!

@PVince81
Copy link
Contributor

How can this even work ? We don't have Webdav PUT upload support in the JS code in 9.1 and from what I remember it was a huge change

@PVince81
Copy link
Contributor

Hmm ok you backported everything... I'm not too happy to backport this huge change and change of behavior of 9.1.

@PVince81
Copy link
Contributor

Had a chat with @DeepDiver1975 and he agreed that we should not backport this. This is a new feature in 10.0 and the code changes are too big.

@DeepDiver1975
Copy link
Member

this is total madness - the feature is not even ready for master/10.0 and we are considering a backport?
this is total nuts - no way I'll ever accept this pr

@butonic
Copy link
Member Author

butonic commented Jan 19, 2017

agreed. backport too risky.

@butonic butonic closed this Jan 19, 2017
@DeepDiver1975 DeepDiver1975 deleted the stable9.1-chunked-web-upload branch August 11, 2017 22:54
@lock
Copy link

lock bot commented Aug 2, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants