Allow configuration of php-http/socket-client >=2.0 with HttplugBundle

[qkdreyer]

Q	A
Bug fix?	yes
New feature?	no
BC breaks?	no
Deprecations?	no
Related tickets	N/A
Documentation	N/A
License	MIT

What's in this PR?

Allow configuration of php-http/socket-client >=2.0 with HttplugBundle

Why?

Configuring php-http/socket-client >=2.0 with HttplugBundle was impossible

Example Usage

N/A

Checklist

• Updated CHANGELOG.md to describe BC breaks / deprecations | new feature | bugfix
• Documentation pull request created (if not simply a bugfix)

To Do

N/A

[qkdreyer]

Ping @joelwurtz @Nyholm

[Nyholm]

This is a BC break. We cannot change constructor arguments without creating a new major version.

Also, I would like to see tests to this PR.

[dbu]

i think we made a mistake with the constructor when creating version 2. in version 1, the signature was responsefactory, config. when moving to version 2, we seem to have wanted to prefer only passing 1 config array, but keep BC with version 1. but we moved the old signature config from 2nd to 3rd parameter, so we lost the config when using the old signature.

to be BC with all cases including the mistake, we should probably in the fallback take $config2 if not null and $config if $config2 is null. the deprecation remains the same.

[dbu]

trying my idea in #65