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.

Sign up for GitHub

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

Jump to bottom

Remove dependency on Guzzle #41

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Remove dependency on Guzzle #41

wants to merge 11 commits into from

Commits on Dec 13, 2016

  1. Update composer to use Httplug adapters 

    `php-http/guzzle-adapter` is still required as dev dependency because it is used as the base adapter. All the unit tests required that client to work.
    @shulard
    shulard committed Dec 13, 2016
    Configuration menu
    Copy the full SHA
    5529906 View commit details
    Browse the repository at this point in the history

  2. Update bindings to use the Http\Client\* objects

    @shulard
    shulard committed Dec 13, 2016
    Configuration menu
    Copy the full SHA
    d2f1c0d View commit details
    Browse the repository at this point in the history

  3. Remove default client creation logic

    @shulard
    shulard committed Dec 13, 2016
    Configuration menu
    Copy the full SHA
    3203897 View commit details
    Browse the repository at this point in the history

  4. Update rawCall request creation logic 

    All the data are computed before `Request` creation. It allow to perform a simple `new Request` with all the analysis result.

Headers are set with authentication, URI is cleaned...
    @shulard
    shulard committed Dec 13, 2016
    Configuration menu
    Copy the full SHA
    3cfd9e4 View commit details
    Browse the repository at this point in the history

  5. Create a central method to send the Request 

    All the calls go through that method, it'll allow extensibility.
    @shulard
    shulard committed Dec 13, 2016
    Configuration menu
    Copy the full SHA
    9f64921 View commit details
    Browse the repository at this point in the history

  6. Update PHPDoc

    @shulard
    shulard committed Dec 13, 2016
    Configuration menu
    Copy the full SHA
    ed48ac6 View commit details
    Browse the repository at this point in the history

  7. Cleanup comments

    @shulard
    shulard committed Dec 13, 2016
    Configuration menu
    Copy the full SHA
    5fbcdf0 View commit details
    Browse the repository at this point in the history

  8. Update ApiTest to use a HttpClient instance 

    Guzzle is still used inside the test logic with middlewares.
    @shulard
    shulard committed Dec 13, 2016
    Configuration menu
    Copy the full SHA
    94de85d View commit details
    Browse the repository at this point in the history

  9. Update FQN reference, don't need to start with \

    @shulard
    shulard committed Dec 13, 2016
    Configuration menu
    Copy the full SHA
    c0d9223 View commit details
    Browse the repository at this point in the history

  10. Update ApiFunctionalTest to use HttpClient

    @shulard
    shulard committed Dec 13, 2016
    Configuration menu
    Copy the full SHA
    cf0ef79 View commit details
    Browse the repository at this point in the history

  11. Add HttpClientDiscovery support 

    Allow to perform a dynamic client resolution based on the current context.
Require the puli composer extension to be registered : https://php-http.readthedocs.org/en/latest/discovery.html#installation
    @shulard
    shulard committed Dec 13, 2016
    Configuration menu
    Copy the full SHA
    139b36d View commit details
    Browse the repository at this point in the history