Last modified: 2014-09-24 01:27:27 UTC
The upload form only checks upload_max_filesize, not post_max_size; it should use the smaller of the two.
Created attachment 6009 [details] A fix
Patch looks okay at a glance. I'd have to test it and switch branches to commit it, though.
Created attachment 6010 [details] A second fix I've corrected the Patch by adding two trim functions before ini_get (line 26-27 of the patch), removing an useless parenthesis on line 29 and removing a double semicolon on line 39.
Applied in r50706. Thanks.
This has regressed somewhere in the meantime; the current code in SpecialUpload.php only checks upload_max_filesize.
Note that UploadFromFile::verifyUpload() does check both upload_max_filesize and post_max_size, but it doesn't seem to trigger properly -- I see no error message instead of seeing a verification failure error. The limits output for the special page UI and the API should check both settings and return the minimum (actually the max file size would be a bit _smaller_ than the max post size, since a little space will be used for other form variables, but that's a start!)
Adding "reviewed" since Stefano's patch was reviewed and, indeed, committed. cc'ing NeilK since he was recently talking/asking about size limits in uploading.
