Skip to content

Administration: Add last fatal PHP error to auto-update failure emails #10445

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 5 commits into
base: trunk
Choose a base branch
from
Open

Administration: Add last fatal PHP error to auto-update failure emails #10445

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
a5e78c5
Administration: Add last fatal PHP error to auto-update failure emails
tlloancy Oct 31, 2025
755902c
Fix: Use = instead of .= to avoid "null" prefix
tlloancy Oct 31, 2025
c36f172
Fix: Read debug.log safely from end, avoid memory crash on large files
tlloancy Nov 1, 2025
e2cd6fd
Fix: Apply WordPress coding standards (PHPCS)
tlloancy Nov 1, 2025
c63ae8c
Final: Full compliance with core review
tlloancy Nov 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
62 changes: 62 additions & 0 deletions src/wp-admin/includes/class-wp-automatic-updater.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1540,6 +1540,16 @@ protected function send_plugin_theme_email( $type, $successful_updates, $failed_
*/
$email = apply_filters( 'auto_plugin_theme_update_email', $email, $type, $successful_updates, $failed_updates );

if ( WP_DEBUG && 'fail' === $type ) {
$fatal_error = get_transient( 'wp_updater_last_fatal_error' );
if ( $fatal_error ) {
$email['body'] .= "\n\n=== " . __( 'LAST FATAL PHP ERROR' ) . " ===\n";
$email['body'] .= '• ' . $fatal_error . "\n";
$email['body'] .= "========================================\n";
delete_transient( 'wp_updater_last_fatal_error' );
}
}

$result = wp_mail( $email['to'], wp_specialchars_decode( $email['subject'] ), $email['body'], $email['headers'] );

if ( $result ) {
Expand Down Expand Up @@ -1828,6 +1838,58 @@ protected function has_fatal_error() {
$result = json_decode( trim( $error_output ), true );
}

if ( WP_DEBUG ) {
$fatal_error = null;

$last_error = error_get_last();
if ( $last_error && in_array( $last_error['type'], array( E_ERROR, E_PARSE, E_COMPILE_ERROR, E_USER_ERROR ) ) ) {
$fatal_error = "PHP Fatal error: {$last_error['message']} in {$last_error['file']} on line {$last_error['line']}";
Copy link
Member

@westonruter westonruter Nov 1, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be translated? I suppose it wouldn't be translated in the error log either.

} else {
$log_file = ini_get( 'error_log' );

if ( ! $log_file ) {
error_log( 'WP Auto-Update: error_log is empty, falling back to debug.log' );
$log_file = WP_CONTENT_DIR . '/debug.log';
}

if ( $log_file && file_exists( $log_file ) && is_readable( $log_file ) ) {
$handle = fopen( $log_file, 'r' );
if ( $handle ) {
$pos = -2;
$current_line = '';
while ( $pos > -filesize( $log_file ) ) {
fseek( $handle, $pos, SEEK_END );
$char = fgetc( $handle );
if ( "\n" === $char ) {
if ( '' !== $current_line ) {
$line = strrev( $current_line );
if ( false !== strpos( $line, 'PHP Fatal error' ) ) {
$fatal_error = trim( $line );
break;
}
$current_line = '';
}
} else {
$current_line .= $char;
}
$pos--;
}
if ( '' !== $current_line && null === $fatal_error ) {
$line = strrev( $current_line );
if ( false !== strpos( $line, 'PHP Fatal error' ) ) {
$fatal_error = trim( $line );
}
}
fclose( $handle );
}
}
}

if ( $fatal_error ) {
set_transient( 'wp_updater_last_fatal_error', $fatal_error, 5 * MINUTE_IN_SECONDS );
}
}

delete_transient( $transient );

// Only fatal errors will result in a 'type' key.
Expand Down
Loading