[wp-trac] [WordPress Trac] #11286: Normal User Input Causes Status 500

[WordPress Trac]

#11286: Normal User Input Causes Status 500
--------------------------+-------------------------------------------------
 Reporter:  miqrogroove   |       Owner:           
     Type:  defect (bug)  |      Status:  new      
 Priority:  normal        |   Milestone:  2.9      
Component:  Comments      |     Version:  2.8.4    
 Severity:  normal        |    Keywords:  has-patch
--------------------------+-------------------------------------------------

Comment(by nacin):

 Replying to [comment:5 miqrogroove]:
 > The server should return OK for the request.

 Bigger picture, we should stop these validation errors from wp_die()-ing
 and instead try to offer some real feedback.

 That's tough to do because it would be a theme-side response -- it sounds
 like a good case of the new theme support registration.

 More immediate picture -- if you're A) omitting fields marked as required,
 B) submitting an invalid email address, or C) submitting an empty comment,
 the client *did* do something wrong, which is why they're getting a wp_die
 error thrown at them. I'm thinking the HTTP response should be a generic
 bad request (400), which is more relevant than 200. I'd find 200 only
 relevant if it wasn't taking the user off to a display-error-and-die page.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/11286#comment:6>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software