Flutterwave / PHP-v3

PHP Library for Flutterwave v3 APIs
MIT License
18 stars 17 forks source link

Please upgrade the Monolog library to version 2.x #13

Closed daigo75 closed 3 years ago

daigo75 commented 3 years ago

The SDK still loads Monolog 1.x, which is an old version of the library, and it can't be used in any environment which requires the newer version. I would recommend to change file composer.json to reference Monolog 2.x and adjust your code to use the new version.

The changes should be minimal, as Monolog 2.x is backward compatible and includes better support for PHP 7. You can find a list of the most important changes here: https://github.com/Seldaek/monolog/blob/master/UPGRADE.md. Such update would allow to use the Flutterwave SDK in modern PHP applications.

bajoski34 commented 3 years ago

@daigo75 this issues has been resolved with latest update to the repo

daigo75 commented 3 years ago

Excellent, thanks 👍

daigo75 commented 3 years ago

Thanks for taking action so quickly. Just a note, if I may. Some customers reported that the latest version of Monolog 2.x requires PHP 7.2. Despite the fact that PHP 5.6 is now obsolete, quite a few servers run that version. Due to that, we modified our solutions to use the latest Monolog 1.x (currently 1.25.5), which works with both PHP 5.6 and 7.x.

I'm aware that this is a "step back", but it was necessary to maintain compatibility with the sites where an old version of PHP is still being used. I thought of letting you know, in case you come across some issues with Monolog 2.x and older PHP versions.