Skip to content
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.

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

Guzzle 6 throwing fatal error when other plugins are using Guzzle 7 #1

Closed
claytoncollie opened this issue Dec 14, 2020 · 7 comments
Closed
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@claytoncollie
Copy link

claytoncollie commented Dec 14, 2020

Hi.

This issue was also raised on wordpress.org.

https://wordpress.org/support/topic/guzzle-6-dependency-throwing-fatal-error/

I have some other plugins that are using Guzzle 7 and it is causing a fatal error with your plugin which is running Guzzle 6.

The proposed solution mentioned here.

guzzle/guzzle#2744 (comment)

@Tropicalista
Copy link
Owner

I know the problem. A lot of plugins are using guzzle.

In the next version I will completely remove Guzzle.

Thanks for reporting.

@Tropicalista Tropicalista self-assigned this Dec 14, 2020
@Tropicalista Tropicalista added enhancement New feature or request good first issue Good for newcomers labels Dec 14, 2020
@Tropicalista
Copy link
Owner

I have scoped all external dependencies. Can you update the plugin and report back if everything is fine?

@claytoncollie
Copy link
Author

I am still getting some conflict with my plugin using Guzzle. It was actually with Google Site Kit that I was requiring with Composer via Wpackagist. I removed Sitekit and all works well now. Thanks for making this update.

Error thrown with message "Call to undefined function Tropicalista\SearchConsole\GuzzleHttp\choose_handler()"

Stacktrace:
#30 Error in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/search-console/vendor/guzzlehttp/guzzle/src/HandlerStack.php:37
#29 Tropicalista\SearchConsole\GuzzleHttp\HandlerStack:create in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/search-console/vendor/guzzlehttp/guzzle/src/Client.php:64
#28 Tropicalista\SearchConsole\GuzzleHttp\Client:__construct in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/search-console/vendor/league/oauth2-client/src/Provider/AbstractProvider.php:114
#27 Tropicalista\SearchConsole\League\OAuth2\Client\Provider\AbstractProvider:__construct in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/search-console/includes/Data/Oauth.php:19
#26 Tropicalista\SearchConsole\Data\Oauth:__construct in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/search-console/includes/Data/Config.php:20
#25 Tropicalista\SearchConsole\Data\Config:__construct in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/search-console/includes/Rest/Rest_Config.php:13
#24 Tropicalista\SearchConsole\Rest\Rest_Config:__construct in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/search-console/includes/REST_API.php:14
#23 Tropicalista\SearchConsole\REST_API:register_routes in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/search-console/includes/REST_API.php:10
#22 Tropicalista\SearchConsole\REST_API:__construct in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/search-console/includes/Plugin.php:191
#21 Tropicalista\SearchConsole\Plugin:init_rest in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-includes/class-wp-hook.php:287
#20 WP_Hook:apply_filters in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-includes/class-wp-hook.php:311
#19 WP_Hook:do_action in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-includes/plugin.php:484
#18 do_action in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-includes/rest-api.php:521
#17 rest_get_server in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-includes/rest-api.php:479
#16 rest_do_request in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-includes/rest-api.php:2520
#15 rest_preload_api_request in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/power-of-attorney/vendor/google-site-kit/includes/Core/Util/BC_Functions.php:37
#14 call_user_func_array in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/power-of-attorney/vendor/google-site-kit/includes/Core/Util/BC_Functions.php:37
#13 Google\Site_Kit\Core\Util\BC_Functions:__callStatic in [internal]:0
#12 array_reduce in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/power-of-attorney/vendor/google-site-kit/includes/Core/Assets/Assets.php:378
#11 Google\Site_Kit\Core\Assets\Assets:Google\Site_Kit\Core\Assets\{closure} in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/power-of-attorney/vendor/google-site-kit/includes/Core/Assets/Script_Data.php:51
#10 call_user_func in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/power-of-attorney/vendor/google-site-kit/includes/Core/Assets/Script_Data.php:51
#9 Google\Site_Kit\Core\Assets\Script_Data:Google\Site_Kit\Core\Assets\{closure} in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/power-of-attorney/vendor/google-site-kit/includes/Core/Assets/Asset.php:108
#8 call_user_func in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/power-of-attorney/vendor/google-site-kit/includes/Core/Assets/Asset.php:108
#7 Google\Site_Kit\Core\Assets\Asset:before_print in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/power-of-attorney/vendor/google-site-kit/includes/Core/Assets/Assets.php:856
#6 Google\Site_Kit\Core\Assets\Assets:run_before_print_callbacks in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/power-of-attorney/vendor/google-site-kit/includes/Core/Assets/Assets.php:865
#5 Google\Site_Kit\Core\Assets\Assets:run_before_print_callbacks in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-content/plugins/power-of-attorney/vendor/google-site-kit/includes/Core/Assets/Assets.php:127
#4 Google\Site_Kit\Core\Assets\Assets:Google\Site_Kit\Core\Assets\{closure} in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-includes/class-wp-hook.php:287
#3 WP_Hook:apply_filters in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-includes/class-wp-hook.php:311
#2 WP_Hook:do_action in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-includes/plugin.php:484
#1 do_action in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-admin/admin-header.php:130
#0 require_once in /Users/claytoncollie/WordPress/contentpilot/app/public/wp-admin/plugins.php:603

@claytoncollie
Copy link
Author

We are still seeing the same problems with Guzzle conflicts.

This time with Simple History and Lara Google Analytics.

https://wordpress.org/plugins/simple-history/

https://wordpress.org/plugins/lara-google-analytics/

@Tropicalista
Copy link
Owner

I will try to reproduce this and will report back.

@Tropicalista
Copy link
Owner

I have tested this and it's working correctly with SiteKit, lara's analytics and simple history. Can you try to uninstall, delete and reinstall search console plugin?
Do you have any cache on the server(litespeed)?

@claytoncollie
Copy link
Author

We are not seeing this issue anymore.

Closing this issue to clean up my dashboard.

Feel free to re-open if interested.

Thanks for all of the feedback and testing those other plugins as well. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants