Skip to content

Allow using any implementation of PSR-7  #453

Description

@shadowhand

Geocoder claims to support PSR-7 but requires a concrete implementation of PSR-7 in egeloen/http-adapter which in turn requires zendframework/zend-diactoros. This is not how PSR-7 is supposed to be used. The only requirement for PSR-7 compatible packages is psr/http-message and all request/response type hints should be against those interfaces.

This currently prevents me from using this package with Guzzle or any other implementation of PSR-7. If I were to install this package right now, I would have both Guzzle and Diactoros installed, implementing the exact same PSR-7 interfaces, which is exactly the situation that PSR-7 is supposed to allow us to avoid.

Activity

  1. changed the title [-]Do not require a specific HTTP adapter[/-] [+]Allow using any implementation of PSR-7 [/+] on Sep 15, 2015
  2. willdurand commented on Oct 9, 2015

    @willdurand
    Member

    This is going to change in the near future. We will adopt HTTPlug

  3. willdurand commented on Oct 9, 2015

    @willdurand
    Member
  4. egeloen commented on Oct 9, 2015

    @egeloen
    Contributor

    I know :) Ivory http adapter is strongly coupled to Diactoros and will not be decoupled from it. Anyway, Httplug is and moving to it should not be so difficult. I will try to provide a PoC this week-end.

  5. bazo commented on Jul 3, 2016

    @bazo

    what's new regarding this issue?
    composer now warns that egeloen/http-adapter is deprecated

  6. willdurand commented on Jul 18, 2016

    @willdurand
    Member

    See #512

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions