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

Fix react/react version for compatibility #23

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Nov 13, 2017

  1. Fix react/react version for compatibility

    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 committed Nov 13, 2017
    Configuration menu
    Copy the full SHA
    7e6dcac View commit details
    Browse the repository at this point in the history