Argument 2 ($url) passed to exclude_rocket_from_wp_updates() must be of the type string

[joejoe04]

Describe the bug
User reported the following error with version 3.17.1 (incomplete stack trace was provided):

Se ha producido un error del tipo E_ERROR en la línea 135 del archivo /home/customer/www/example.net/public_html/wp-content/plugins/wp-rocket/inc/Engine/Plugin/UpdaterSubscriber.php. Mensaje de error: Uncaught TypeError: Argument 2 passed to WP_Rocket\Engine\Plugin\UpdaterSubscriber::exclude_rocket_from_wp_updates() must be of the type string, null given, called in /home/customer/www/example.net/public_html/wp-includes/class-wp-hook.php on line 324 and defined in /home/customer/www/example.net/public_html/wp-content/plugins/wp-rocket/inc/Engine/Plugin/UpdaterSubscriber.php:135 Stack trace: #0 /home/customer/www/example.net/public_html/wp-includes/class-wp-hook.php(324): WP_Rocket\Engine\Plugin\UpdaterSubscriber->exclude_rocket_from_wp_updates(Array, NULL) #1 /home/customer/www/example.net/public_html/wp-includes/plugin.php(205): WP_Hook->apply_filters(Array, Array) #2 /home/customer/www/example.net/public_html/wp-includes/class-wp-http.php(234): apply_filters('http_request_ar...', Array, NULL) #3 /home/customer/www/example.net/public_html/wp-includes/class-wp-http.php(638): WP_Http->request(NULL, Array) #4 /home/customer/www/example.net/public_html/wp-includes/http.php(184): WP_Http->get(NULL, Array) #5 /h

It relates to this line:

wp-rocket/inc/Engine/Plugin/UpdaterSubscriber.php

Line 135 in ce7b601

public function exclude_rocket_from_wp_updates( $request, string $url ) {

I can see a recent update to the related method here:
7348869
https://imageshack.com/a/img922/4788/YNsWPZ.png

The change removed the check to make sure $url is a string, and it opens us up to errors like these.

To Reproduce
Steps to reproduce the behavior:

1. Update to version 3.17.1
2. Not sure exactly what other condition is required. Could be a conflict with another plugin or theme.
3. Anything that could cause the $url parameter to not be of type string.

Expected behavior
We should check to confirm the $url parameter is a string to prevent this error from happening.

Additional context
Ticket - https://secure.helpscout.net/conversation/2737001255/518317
Slack - https://wp-media.slack.com/archives/C43T1AYMQ/p1729176825682779

[joejoe04]

Related: https://secure.helpscout.net/conversation/2737958899/518476/

[piotrbak]

The complete stack trace is in the ticket above

[alfonso100]

related: https://secure.helpscout.net/conversation/2739225788/518711?folderId=2683093

[alfonso100]

another one: https://secure.helpscout.net/conversation/2740067797/518850?folderId=2008123

error stack

/public/wp-content/plugins/wp-rocket/inc/Engine/Plugin/UpdaterSubscriber.php on line 135
[21-Oct-2024 08:37:27 UTC] PHP Fatal error: Uncaught TypeError: WP_Rocket\Engine\Plugin\UpdaterSubscriber::exclude_rocket_from_wp_updates(): Argument #2 ($url) must be of type string, null given, called in /public/wp-includes/class-wp-hook.php on line 324 and defined in /public/wp-content/plugins/wp-rocket/inc/Engine/Plugin/UpdaterSubscriber.php:135
Stack trace:
#0 /public/wp-includes/class-wp-hook.php(324): WP_Rocket\Engine\Plugin\UpdaterSubscriber->exclude_rocket_from_wp_updates()
#1 /public/wp-includes/plugin.php(205): WP_Hook->apply_filters()
#2 /public/wp-includes/class-wp-http.php(234): apply_filters()
#3 /public/wp-includes/class-wp-http.php(638): WP_Http->request()
#4 /public/wp-includes/http.php(184): WP_Http->get()
**#5 /public/wp-content/plugins/bizreview-pro/inc/google-review/google-api-config.php(50): wp_remote_get()**
#6 /public/wp-content/plugins/bizreview-pro/inc/functions.php(28): BizReview_Google_API->get_place_data()
#7 /public/wp-includes/class-wp-hook.php(324): bizreview_google_api()

[UpworksTeam]

The same thing here on 3 websites after updating from 3.17.0.2 to 3.17.1

[viobru]

Related: https://secure.helpscout.net/conversation/2736553837/518231

[alfonso100]

related:
https://secure.helpscout.net/conversation/2742392632/519264?folderId=273761
https://secure.helpscout.net/conversation/2739071952/518691

[Khadreal]

I think we have few instances where $url is not set.

Scope a solution ✅

We can add a nullable type hint to the ?string $url here and a condition to check if $url is not string

if( is_null( $url ) ) {
    return $request;
}

Estimate the effort ✅

[XS]

[remyperona]

IMO, we should revert to what was there before: no typehint in the method declaration, and a check for is_string() inside the method. We can't know if a plugin will be modifying the URL value to a specific type, some plugins use null, others might use false.

[joejoe04]

Related: https://secure.helpscout.net/conversation/2743534579/519516