[wp-trac] Re: [WordPress Trac] #7890: Internal server error when submitting comment without name or email

[WordPress Trac]

#7890: Internal server error when submitting comment without name or email
-------------------------------------------------------+--------------------
 Reporter:  gadlen                                     |        Owner:  Libor Jelinek
     Type:  defect                                     |       Status:  assigned     
 Priority:  normal                                     |    Milestone:  2.7          
Component:  Comments                                   |      Version:  2.6.1        
 Severity:  normal                                     |   Resolution:               
 Keywords:  2nd-opinion dev-feedback has-patch commit  |  
-------------------------------------------------------+--------------------
Changes (by DD32):

  * keywords:  2nd-opinion, dev-feedback => 2nd-opinion dev-feedback has-
               patch commit

Comment:

 my vote: Keep it as 500, but return a >512 byte error. a 500char comment
 on the wp_die() page would suffice, The attached patch of mine works, and
 only adds the ammount of spaces needed, but theres no reason it couldnt be
 simplified to just dump 500 spaces onto the page.

 Its too late to change the error HTTP status type, but something needs to
 happen to prevent IE being IE, Regardless of your(refering to all
 readers), the wp_die() function needs to support sending of 500 errors
 (Primarily due to 200 errors being cached by search engines and the alike,
 which was the original intention of the 500 error IIRC) as thats what its
 been doing.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/7890#comment:15>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software