fix wrong default settings passing

[VoDmAl]

No description provided.

[JimmyRittenborg]

Sorry can you elaborate more on this one, maybe leave en example? ☺️

[VoDmAl]

Hi @JimmyRittenborg,

Yep. I found mistake with passing default settings as nested array with key "defaults". As far as i can see it is not used in current versions of guzzlehttp (downloaded via composer as the project's dependency).
May be this is the reason why I see duplicate of passing "allow_redirects" in each request in "doRequest"
$requestOptions = array( ... 'allow_redirects' => false, ... );
I found this bug when tried to pass 'verify' => false and used getClient() as example to create my own instance.
This will not work with my self-signed certificate site:
$client->setClient(new \GuzzleHttp\Client(['defaults' => ['allow_redirects' => false, 'cookies' => true, 'verify' => false]]));
This will work:
$client->setClient(new \GuzzleHttp\Client(['allow_redirects' => false, 'cookies' => true, 'verify' => false]));

[JimmyRittenborg]

Thank you for describing this. Though this repo is just a fork of the original https://github.com/FriendsOfPHP/Goutte but makes it possible to use Goutte with the FileCookieJar from Guzzle. So if you don't need that, you could just pull the original repo and maybe make the PR over there? :)

[VoDmAl]

Yep. But I took your fork as it is ahead of original. And the original repo does not look alive now. A bunch of opened pull requests. :)