Why ringcentral/psr7 and not guzzle/psr7 #331

Activity

1. jsor commented on Nov 13, 2018

   

   jsor

   on Nov 13, 2018

   Member

   More actions

   See #319
2. phpbg commented on Nov 13, 2018

   

   phpbg

   on Nov 13, 2018

   Author

   More actions

   Mmmhh ok, sorry for the duplicate.
   Don't you want to update the minimum requirements for react/http ?
   5.3 is quite obsolete now...
   Regards
3. clue

   added

   question

   on Nov 13, 2018
4. clue commented on Nov 13, 2018

   

   clue

   on Nov 13, 2018

   Member

   More actions

   "Why guzzle/psr7 and not ringcentral/psr7?" :-)

   I would argue that it doesn't really make much of a difference which PSR-7 implementation we're using. For the most part, this is an internal implementation detail that is not in any way exposed to the outside and our documentation explicitly says we're using PSR-7 interfaces only.

   While I can see where you're coming from, I don't think discussing PHP version requirements adds much to this specific discussion.

   Additionally, there are plans to drop RingCentral in a future version. Partly because it adds a lot of stuff that we don't really use here (sync file handling etc.) and partly because it's missing some stuff that we do need here (async implementations etc.).

   I believe this has been answered, so I'm closing this for now. Please come back with more details if this problem persists and we can reopen this 👍

   😕React with 😕1Reacted by Valithor Obsidion
5. clue

   closed this as completedon Nov 13, 2018