Changeset 7292 for sites/trunk/wordpress.org/public_html/wp-content/plugins/official-wordpress-events/official-wordpress-events.php

Timestamp:

06/11/2018 11:25:40 PM (8 years ago)

Author:

coreymckrill

Message:

Official WordPress Events: More info on remote_get error

The remote_get method is occasionally returning a 400 error, but the response
body seems to be empty. This adds the response message to the error report
in the hopes that it will provide a clue as to why these are happening.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/official-wordpress-events/official-wordpress-events.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/official-wordpress-events/official-wordpress-events.php

@@ -628 +628 @@
                         $this->maybe_pause( wp_remote_retrieve_headers( $response ) );
 
+                        $response_code    = wp_remote_retrieve_response_code( $response );
+                        $response_message = wp_remote_retrieve_response_message( $response );
+                        $response_body    = wp_remote_retrieve_body( $response );
+
                         if ( is_wp_error( $response ) ) {
                                 $error_messages = implode( ', ', $response->get_error_messages() );
@@ -639 +643 @@
                                         );
                                 }
-                        } elseif ( 200 != $response['response']['code'] ) {
+                        } elseif ( 200 != $response_code ) {
                                 // trigger_error() has a message limit of 1024 bytes, so we truncate $response['body'] to make sure that $body doesn't get truncated.
-
                                 $error = sprintf(
-                                        'Received HTTP code: %s and body: %s. Request was to: %s; Arguments were: %s',
-                                        $response['response']['code'],
-                                        substr( sanitize_text_field( $response['body'] ), 0, 500 ),
+                                        "HTTP Code: %d\nMessage: %s\nBody: %s\nRequest URL: %s\nArgs: %s",
+                                        $response_code,
+                                        sanitize_text_field( $response_message ),
+                                        substr( sanitize_text_field( $response_body ), 0, 500 ),
                                         $url,
                                         print_r( $args, true )
                                 );
 
-                                $response = new WP_Error( 'woe_invalid_http_response', 'Invalid HTTP response code', $response );
+                                $response = new WP_Error( 'owe_invalid_http_response', 'Invalid HTTP response code', $response );
                         }
 
                         if ( $error ) {
                                 $error = preg_replace( '/&key=[a-z0-9]+/i', '&key=[redacted]', $error );
-                                trigger_error( sprintf( '%s error for %s: %s', __METHOD__, parse_url( site_url(), PHP_URL_HOST ), sanitize_text_field( $error ) ), E_USER_WARNING );
+                                trigger_error( sprintf(
+                                        '%s error for %s: %s',
+                                        __METHOD__,
+                                        parse_url( site_url(), PHP_URL_HOST ),
+                                        sanitize_text_field( $error )
+                                ), E_USER_WARNING );
 
                                 $to = apply_filters( 'owe_error_email_addresses', array() );
                                 if ( $to && ( ! defined( 'WPORG_SANDBOXED_REQUEST' ) || ! WPORG_SANDBOXED_REQUEST ) ) {
-                                        wp_mail( $to, sprintf( '%s error for %s', __METHOD__, parse_url( site_url(), PHP_URL_HOST ) ), sanitize_text_field( $error ) );
+                                        wp_mail(
+                                                $to,
+                                                sprintf(
+                                                        '%s error for %s',
+                                                        __METHOD__,
+                                                        parse_url( site_url(), PHP_URL_HOST )
+                                                ),
+                                                sanitize_text_field( $error )
+                                        );
                                 }
                         }