search

optimizely / php-sdk

PHP SDK for Optimizely Feature Experimentation and Optimizely Full Stack (legacy)
https://docs.developers.optimizely.com/experimentation/v4.0.0-full-stack/docs/php-sdk
Apache License 2.0
18 stars 29 forks source link

PHP 7.4 support (upgrade monolog, dependencies) #191

Closed d8vjork closed 2 years ago

d8vjork commented 4 years ago

Hi there 🙋‍♂

Tests seems to fail on PHP 7.4, 7.3 still fine though.

Another problem that I don't know is that if this package will upgrade its dependencies like Monolog, because I'm writing a wrapper/integration with Laravel for one of my projects so it requires monolog 2.x from Laravel and so it has dependencies conflicts (specially with monolog)

Afrowson commented 4 years ago

I am a Software developer at www.aboalarm.de This is also a major issue for us. We want to use optimizely, but our internal logging and Laravel use monolog/monolog 2.0.

Please fix this asap :)

d8vjork commented 4 years ago

@Afrowson I was also looking into the same thing creating a package to bring this to Laravel

jeffochoa commented 4 years ago

✋ facing the same issue here. Laravel user as well.

fbnfgc commented 4 years ago

Any update about PHP 7.4 support? As mentionned by @d8vjork, this package supports 5.5, 5.6, 7.0 and 7.1 which are out of support.

PHP 7.4 has been released at the end of 2019. Four months ago PHP 7.4 represented 20% of the projects.

Have you considered bumping a major version to drop old PHP versions?

Thanks!

jjmu15 commented 3 years ago

I desperately want to use this in my Laravel application also but have the same problem. Any chance of this getting actioned soon as it seems to have been stagnant for a long time? :(

that-guy-iain commented 3 years ago

@optimizely/fullstack-devs (aww I can't tag them all). OK @yavorona @oakbani @jaeopt since you guys have worked on it this year and this ticket is from last year.

Currently, 7.4 is ~46% and 8.0 is ~18% of the usage. People literally can't use this because you haven't upgraded the libraries to be usable on the latest versions.

There is no point in adding features to an unusable library.

zashraf1985 commented 2 years ago

We just removed the restriction of monolog version from the sdk. We have tested it with php 8.1.2 along with monolog version 2.3 and it appears to be working. Please update PHP SDK version to 3.9.0 and it should work.