Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace guzzle with httplug #77

Merged
merged 13 commits into from Nov 24, 2017
Merged

Conversation

veewee
Copy link
Contributor

@veewee veewee commented Sep 29, 2017

fixes #25

@veewee veewee self-assigned this Sep 29, 2017
@veewee
Copy link
Contributor Author

veewee commented Oct 27, 2017

Upgrading:

composer require psr/http-message:^1.0 php-http/httplug:^1.1 php-http/message-factory:^1.0 php-http/discovery:^1.3 php-http/message:^1.6 php-http/client-common:^1.6 php-http/guzzle6-adapter:^1.1
// Wrap your guzzle client in a HTTPlug adapter:
$httpClient = new Client($guzzleClient);

// Use the new HttPlugHandle instead of the old GuzzleHandle
$clientBuilder->withHandler(HttPlugHandle::createForClient($httpClient));

// use the new HttPlugWsdlProvider instead of the old GuzzleWsdlHandle
$clientBuilder->withWsdlProvider(HttPlugWsdlProvider::createForClient($httpClient));

// If you are using NTLM:
// Remove the NtlmMiddleware and add the curl options to the guzzle client configuration instead:
$httpClient = Client::createWithConfig(['curl' => [
    CURLOPT_HTTPAUTH => CURLAUTH_NTLM,
    CURLOPT_USERPWD => 'username:password',
]]);

@veewee veewee changed the title WIP Replace guzzle with httplug Replace guzzle with httplug Oct 27, 2017
@veewee veewee added this to the 0.6.0 milestone Oct 27, 2017
@janvernieuwe janvernieuwe merged commit cc15a60 into phpro:master Nov 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use the http-interop layer instead of Guzzle.
2 participants