WordPress.org

Make WordPress Core

Opened 7 years ago

Last modified 3 years ago

#18474 reopened defect (bug)

Misleading error message when theme ZIP exceeds post_max_size

Reported by: ocean90 Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 3.2
Component: Upload Keywords: has-patch dev-feedback
Focuses: administration Cc:

Description

post_max_size is 32MB, now try to uploading a 40MB big ZIP.

You will get the Are you sure you want to do this? Please try again. message. But try again will not help.

Notice: Warning: POST Content-Length of 47774864 bytes exceeds the limit of 33554432 bytes in Unknown on line 0

Attachments (1)

18474.diff (1006 bytes) - added by kurtpayne 7 years ago.
Patch to detect truncated POST requests

Download all attachments as: .zip

Change History (7)

@kurtpayne
7 years ago

Patch to detect truncated POST requests

#1 @kurtpayne
7 years ago

  • Cc kpayne@… added
  • Component changed from Themes to Upload
  • Keywords has-patch dev-feedback added; needs-patch removed

This happens when php stops listening to the request after post_max_size is reached. When that happens, the mime request cannot be properly decoded, so no post variables come through (including the nonce).

18474.diff will detect the problem when a nonce fails and will give the proper error message, but there's nothing we can do about it from within WP.

#2 follow-up: @iseulde
5 years ago

  • Keywords has-patch dev-feedback removed
  • Resolution set to invalid
  • Status changed from new to closed

Tested in 3.6 and get an appropriate error: test.zip exceeds the maximum upload size for this site. Maximum upload file size: 32MB.

#3 in reply to: ↑ 2 @helen
5 years ago

Replying to avryl:

Tested in 3.6 and get an appropriate error: test.zip exceeds the maximum upload size for this site. Maximum upload file size: 32MB.

That sounds like a multisite setting, as opposed to post_max_size.

#4 @helen
5 years ago

  • Resolution invalid deleted
  • Status changed from closed to reopened

#5 @iseulde
5 years ago

  • Keywords has-patch dev-feedback added

Sorry, should have read on.

#6 @chriscct7
3 years ago

  • Focuses administration added
Note: See TracTickets for help on using tickets.