[wp-trac] [WordPress Trac] #62029: Do not bloat Debug Log with Scraping home page… when auto upgrade is OK

[WordPress Trac]

#62029: Do not bloat Debug Log with Scraping home page… when auto upgrade is OK
-----------------------------+---------------------
 Reporter:  georgwordpress   |       Owner:  (none)
     Type:  enhancement      |      Status:  new
 Priority:  normal           |   Milestone:  6.7
Component:  Upgrade/Install  |     Version:
 Severity:  normal           |  Resolution:
 Keywords:                   |     Focuses:
-----------------------------+---------------------
Changes (by SergeyBiryukov):

 * component:  General => Upgrade/Install
 * milestone:  Awaiting Review => 6.7


Comment:

 Hi there, welcome to WordPress Trac! Thanks for the ticket.

 Looking at [58128] / #58281, it appears that the intention was to output
 `true` in the error log in case of no fatal errors:
 {{{
 // If this outputs `true` in the log, it means there were no fatal errors
 detected.
 if ( $is_debug ) {
         error_log( var_export( substr( $response['body'], strpos(
 $response['body'], '###### wp_scraping_result_start:' ) ), true ) );
 }
 }}}

 However, the current code can indeed output the whole HTML body if `######
 wp_scraping_result_start:` is not found. Moving to 6.7 for investigation.

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