search

jenssegers / php-proxy

A PHP proxy script with https and post support
https://jenssegers.com
933 stars 266 forks source link

Update for newer PHP and dependencies #82

Open Synchro opened 4 years ago

Synchro commented 4 years ago

This PR makes the test suite run (successfully) on PHP 7.3. and 7.4, raises the minimum to PHP 7.3 (the oldest currently supported release), and updates various dependencies for more recent versions, in particular PHPUnit 9 and Guzzle 7.

I tried it on PHP 8.0 but it breaks because there is no compatible version of laminas/laminas-diactoros on 8.0 yet so I left that out of this PR, but at least you know where the problems lie.

If this is merged and tagged, I recommend bumping the major version since these updates amount to a BC break.

edurenye commented 3 years ago

now laminas/laminas-diactoros ^2.5 supports PHP 8

Synchro commented 3 years ago

I have bumped diactoros to 2.5 and added PHP 8.0 compat in composer, and it all works very nicely, so thanks for the heads-up @edurenye.

I also bumped some dev dependencies (inc PHPUnit 9.5) and added the roave/security-advisories package. All tests still pass, including on PHP 8.1.

edurenye commented 3 years ago

This other PR also updates this stuff: https://github.com/jenssegers/php-proxy/pull/90, But this issue is older, so I think this PR should be kept