Fix content deflating for IIS HTTP responses

[blat]

Q	A
Bugfix	yes
BC Break	no

Description

Some response coming from IIS are using gzuncompress instead of gzinflate.

Note: This fix was already fixed 10 years ago on ZF1 zendframework/zf1@1e158aa but this this was not ported in ZF2.
See: https://framework.zend.com/issues/browse/ZF-12457.html

To reproduce:

use Laminas\Http\Client;
$client = new Client('https://www.vnews.com/With-Federal-Money-Running-Out-Lebanon-Starts-to-Pick-Up-Motel-Tabs-for-Homeless-48667855');
echo $client->send()->getBody();

Current result:

PHP Warning: gzuncompress(): data error in vendor/laminas/laminas-http/src/Response.php on line 653

[froschdesign]

@blat
You are using the wrong branch because 2.17.0 is the last release, not 2.16.
See also in the contributing guidelines: What branch to issue the pull request against?

[blat]

@froschdesign

PHP7 is not supported by 2.17.x release.
No plan for a 2.16.1?

[froschdesign]

PHP7 is not supported by 2.17.x release.

Why? PHP 7 is dead.

[blat]

Yes, I know PHP7 will be EOL at the end of the month. I'm current working on this upgrade ;)
In the meantime, a 2.16.1 will be appreciated!

[Ocramius]

Some minor adjustments needed.

@froschdesign I'd be OK with merging this to an old-stable, as an exception: the contributor has put time to fix in it, and the merge-up should be straightforward too. WDYT?

[froschdesign]

@Ocramius

…the contributor has put time to fix in it, and the merge-up should be straightforward too.

I agree with that. 👍🏻

[blat]

I've updated with the smaller example I was able to found.

Thank you!

[Ocramius]

Thanks @blat! Releasing now :)