Skip to content
This repository has been archived by the owner on Sep 23, 2021. It is now read-only.

Dependance avec reactphp/react #21

Open
ChoOo7 opened this issue Oct 10, 2017 · 2 comments
Open

Dependance avec reactphp/react #21

ChoOo7 opened this issue Oct 10, 2017 · 2 comments

Comments

@ChoOo7
Copy link

ChoOo7 commented Oct 10, 2017

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 !

@omansour
Copy link
Contributor

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

merci

@ChoOo7
Copy link
Author

ChoOo7 commented Nov 13, 2017

Voila qui est fait :)
#23
merci

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants