justbetter / magento2-sentry

Magento 2 module to log to Sentry
MIT License
165 stars 70 forks source link

Incompatible with monolog/monolog v2.7.0 #93

Closed adamj88 closed 2 years ago

adamj88 commented 2 years ago

Today, monolog/monolog was updated to v2.7.0 where they added an additional $datetime parameter to the addRecord method in: https://github.com/Seldaek/monolog/releases/tag/2.7.0

Since this module requires >=2.0.0 of monolog an error is produced:

Warning: Declaration of JustBetter\Sentry\Plugin\MonologPlugin::addRecord(int $level, string $message, array $context = Array): bool should be compatible with Monolog\Logger::addRecord(int $level, string $message, array $context = Array, ?Monolog\DateTimeImmutable $datetime = NULL): bool in vendor/justbetter/magento2-sentry/Plugin/MonologPlugin.php on line 54

Ref:

indykoning commented 2 years ago

This has now been fixed and released as version 3.1.0 🚀