Commit 6605872 changed chunk_size from 1mb fixed size to use php's post_max_size setting. There is probably some overhead, so posts are now larger than post_max_size when uploading a file that needs chunking.

The solution would be to either reduce the chunk_size by a couple of kb to accommodate this overhead, or - just leave this at 1mb. This seems like a sane default and should not have any performance impact. Chunks are also needed for the Flash uploader for progress bar. It only updates for each chunk, so needs smaller chunks to be more interactive (and is said to be more stable using chunks).

Besides, module developers implementing this field can set their own chunk_size if needed by adding e.g. '#plupload_settings' => array('chunk_size' => '512kb') to their field instance.

Atle

CommentFileSizeAuthor
#1 set-chunk_size-to-1mb-1461884-1.patch749 bytesatlea
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

atlea’s picture

Here's a patch for 1mb fixed size for the people out there experiencing problems and needing a fix now. :)

slashrsm’s picture

Status: Active » Fixed

That makes sense. Commited to 7.x-1.x: http://drupalcode.org/project/plupload.git/commit/3ef7aa9

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit 3ef7aa9 on 7.x-1.x, 7.x-2.x, 8.x-1.x by slashrsm:
    Issue #1461884 by atlea: File uploads larger than php post_max_size...
bwaindwain’s picture

Issue summary: View changes

To anyone interested, chunk_size setting is changed again later in https://www.drupal.org/node/1230632