[wp-trac] [WordPress Trac] #28319: Add hook to WP_Error for logging and report purposes

[WordPress Trac]

#28319: Add hook to WP_Error for logging and report purposes
-----------------------------------------+-----------------------------
 Reporter:  ebinnion                     |       Owner:  johnbillion
     Type:  enhancement                  |      Status:  reopened
 Priority:  normal                       |   Milestone:  Future Release
Component:  General                      |     Version:  trunk
 Severity:  normal                       |  Resolution:
 Keywords:  2nd-opinion has-patch close  |     Focuses:
-----------------------------------------+-----------------------------
Changes (by rmccue):

 * keywords:  2nd-opinion has-patch => 2nd-opinion has-patch close


Comment:

 Replying to [comment:17 danielbachhuber]:
 > I disagree. Creating a `WP_Error` object is a deliberate decision of the
 developer to report an error upstream. Returning false is for a valid
 condition.

 FWIW, I agree with nacin. The REST API will make pretty liberal use of
 WP_Error (intentionally), and logging all of those alone would be
 problematic. In addition, there's the possibility for a DoS attack using
 it if errors are logged to a DB.

 Recommend close as wontfix.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/28319#comment:20>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform