Upgrade Plupload to 2.0

[azaozz]

Plupload 2.0 was ​released about a month ago. Has quite a bit of fixes and improvements. The upgrade should be a straight replacement.

[azaozz]

Consider removing the Silverlight component. It is useful only for IE < 10 when there's no flash, which seems quite rare.

[ocean90]

Since the current uploader isn't much broken we can punt this for now.

Just an info: These are the files of 2.0.0:
moxie.js
moxie.min.js
Moxie.swf
Moxie.xap
plupload.dev.js
plupload.full.min.js
plupload.min.js
license.txt

And these are the current ones from core:
changelog.txt
license.txt
plupload.flash.js
plupload.flash.swf
plupload.html4.js
plupload.html5.js
plupload.js
plupload.silverlight.js
plupload.silverlight.xap

[azaozz]

In 25663.patch:

• Plupload 2.1.1.
• Remove support for Silverlight. Flash is used in IE < 10. If flash is not installed, it will fall back to html4.
• Remove the limitation and warnings for the flash runtime when uploading files larger than 100MB. Works well now.

[kovshenin]

Refreshed in 25663.diff​, added Silverlight back together with Moxie.xap. The patch contains binary files as well as file deletions, should apply cleanly with svn patch path/to/file.diff.

Tested the html5, flash and html4 runtimes in Chrome on OS X, looks good. The flash runtime doesn't work in Safari and Firefox, still trying to debug why. Haven't tested Windows, IE or the silverlight runtime yet. Any help appreciated.

[azaozz]

In 27316:

Upgrade Plupload to 2.1.1, props kovshenin, see #25663

[azaozz]

In 27318:

Remove debugging cruft, see #25663

[TobiasBg]

The moxie.xap has svn:executable set, so that file rights change from 644 to 755.
Not sure if this is necessary? Could that maybe cause problems with automatic updates?

[azaozz]

In 27340:

Plupload: remove the (old) plupload.silverlight.xap, remove the "executable" prop from Moxie.xap, see #25663

[kovshenin]

Looked deeper into the Flash runtime issue in Firefox, Safari. It looks like Moxie.swf doesn't pass any cookies to async-upload.php so that's responding with a redirect. It works fine in Chrome and IE.

Been digging around the Plupload/Moxie forums and repo, but couldn't find much related. Closest I got was this 4yr old ​issue, where a suggested workaround was to include the session hash in the request. But then again everything worked well in 1.5.7 so I'm guessing it's a bug upstream with the change to Moxie.swf.

To work around this we can partially revert r18674 where we stopped passing our authentication cookies to post_params. The checks for them are still in place in async-upload.php, but need a small tweak to support the upload-attachment action (background: r22902).

Patch in 25663.2.diff​.

[azaozz]

Replying to kovshenin:

To work around this we can partially revert r18674 where we stopped passing our authentication cookies to post_params.

Not sure this is a good idea. Our cookies are HTTP only, exposing them in the HTML is not good. That was one of the reasons for switching to Plupload too.

Emailed Davit (the lead developer of Plupload) and he traced the problem to the removal of the urlstream_upload option from 2.0. This was setting alternate mode in the flash runtime which ensures sending of cookies but requires the whole file to be loaded in memory. This also prevents uploading of big files with flash.

The same can be done in 2.0 by using

required_features: {
     send_binary_string: true
}

Tested further and it seems flash always sends cookies in IE, regardless of the above setting. 25663.3.diff is based on that. It checks whether flash will be used, then sets the above option in non IE browsers. As by default we use flash only for old IE, this seems the best alternative.

More testing especially in IE < 10 very welcome :)

[azaozz]

In [27662]:

Plupload: switch to urlstream upload method when the 'flash' runtime is used in non IE browsers. This ensures cookies are sent but limits the maximum file size that flash can handle.

By default only IE9 and older use flash so this change is mostly for completeness. It would affect only the (extremely rare) cases where a plugin disables the html5 runtime.

[azaozz]

[27662] changes the runtime order to html5,flash,silverlight,html4 which is the default in Plupload. That used to be html5,silverlight,flash,html4 as flash has problems with big files when using 'urlstream_upload' or send_binary_string: true. Without that setting (currently default for IE) flash seems to perform better than silverlight.

Lets test this for few days and revert it if there are any problems. Testing big files (over 200MB) in old IE would be best.

[kovshenin]

I tested IE9 and IE8 with Flash and a 500MB file and it worked like a charm. Like a slow charm.

[nacin]

Either re-open this ticket or create new tickets for additional issues.