PhpProcessManagerBundle icon indicating copy to clipboard operation
PhpProcessManagerBundle copied to clipboard

Dependance avec reactphp/react

Open ChoOo7 opened this issue 7 years ago • 2 comments

Bonjour

Je pense qu'il y a une dépendance cassée avec reactphp

Je ne sais pas si je m'exprime bien mais si l'on compare la version 0.4.4 de reactphp et la version 0.4.1 0.4.4 : https://github.com/reactphp/http/blob/v0.4.4/src/Server.php#L68

0.4.1 : https://github.com/reactphp/http/blob/v0.4.1/src/Server.php#L61

Le emit('data' sur la request est envoyé seulement lorsque le boddy buffer est non vide (et donc pas tout le temps, pas lors d'une simple requete GET)

Hors dans https://github.com/M6Web/PhpProcessManagerBundle/blob/master/Bridge/HttpKernel.php#L48

Le parti est pris que l'on a systématiquement cet evenement

Donc cela ne marche plus

Si dans votre composer.json on fixe un dependance vers reactphp/react non pas à

    "react/react"                    : "^0.4"

Mais à

    "react/react"                    : "0.4.1"

Il ne devrait plus y avoir de soucis.

Merci pour votre temps

Please, make my application superflight again !

ChoOo7 avatar Oct 10 '17 14:10 ChoOo7

hey - ça me semble correct ! possible de proposer une PR rapide autour de ça ?

merci

omansour avatar Oct 10 '17 16:10 omansour

Voila qui est fait :) https://github.com/M6Web/PhpProcessManagerBundle/pull/23 merci

ChoOo7 avatar Nov 13 '17 09:11 ChoOo7