search

stayallive / wp-sentry

A (unofficial) WordPress plugin reporting PHP and JavaScript errors to Sentry.
https://wordpress.org/plugins/wp-sentry-integration/
MIT License
300 stars 48 forks source link

PHP 8.3 warning #191

Closed rafaucau closed 2 months ago

rafaucau commented 3 months ago

This warning is displayed when WP_DEBUG_DISPLAY is enabled:

Deprecated: Creation of dynamic property WPSentry\ScopedVendor\GuzzleHttp\Psr7\CachingStream::$stream is deprecated in /*****/web/app/plugins/wp-sentry-integration/build/vendor/guzzlehttp/psr7/src/CachingStream.php on line 28

dingo-d commented 2 months ago

Same for me, but this is not the fault of the plugin, but the SDK (well, the Guzzle PSR7 dependency actually).

EDIT: @stayallive maybe if you'd require a newer version of guzzlehttp/psr7 (^2.1.1) in your package, that would download the newer version (currently it's 1.9.1).

stayallive commented 2 months ago

This issue is:

guzzlehttp/psr7 2.6.2 requires php (^7.2.5 || ^8.0 but 7.2.0 is installed)

I guess we can bump our minimum PHP version to 7.2.5 as well instead of 7.2.0.

dingo-d commented 2 months ago

I don't think it can hurt 👍🏼

stayallive commented 2 months ago

https://github.com/stayallive/wp-sentry/releases/tag/v7.15.0 was just released which updates guzzlehttp/psr7. This should resolve the issue, feel free to re-open if not.