Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 16:10
    CharlotteDunois closed #211
  • 16:10
    CharlotteDunois closed #143
  • 16:10
    CharlotteDunois closed #69
  • 16:10
    CharlotteDunois closed #199
  • 14:43

    WyriHaximus on master

    Add .gitattributes to exclude d… Merge pull request #72 from ree… (compare)

  • 14:43
    WyriHaximus closed #72
  • Dec 03 18:15
    WyriHaximus review_requested #154
  • Dec 03 16:46

    WyriHaximus on 2.x

    Add .gitattributes to exclude d… Merge pull request #154 from re… (compare)

  • Dec 03 16:46
    WyriHaximus closed #154
  • Dec 03 07:31
    WyriHaximus synchronize #214
  • Dec 03 07:30

    jsor on master

    Fix handling of multiple cookie… Merge pull request #352 from fi… (compare)

  • Dec 03 07:30
    jsor closed #352
  • Dec 02 23:59
    Bilge commented #154
  • Dec 02 23:59
    Bilge commented #154
  • Dec 02 19:28

    jsor on master

    Fix Travis CI builds, do not in… Merge pull request #202 from cl… (compare)

  • Dec 02 19:28
    jsor closed #202
  • Dec 02 19:25
    WyriHaximus commented #153
  • Dec 02 19:23
    reedy commented #153
  • Dec 02 19:21
    WyriHaximus commented #153
  • Dec 02 19:11
    clue review_requested #202
Christian Lück
@clue
Like @WyriHaximus said, we're going to improve this API wise and likely offer dedicated overwrite/append methods in the upcoming version
Marc Morera
@mmoreram
great so much @clue
Christian Lück
@clue
Yey, StickerMule is launching a promo again, some ReactPHP stickers will be shipped soon :+1:
Cees-Jan Kiewiet
@WyriHaximus
:tada:
Cees-Jan Kiewiet
@WyriHaximus
@clue thanks for your review on reactphp/promise#150 will fix and respond later. Essentially the \METHOD will get a follow up PR anyway to use use function METHOD; to achive the same effect but cleaner
Cees-Jan Kiewiet
@WyriHaximus
@clue ugh this cherry-picking is a bit more complicated then I hoped xD
Christian Lück
@clue
@WyriHaximus Let me know if there's anything I can help with :+1:
Cees-Jan Kiewiet
@WyriHaximus
@clue will do, essentially I'm missing cancelRequests on the promise object in master. So need to figure out hwo to handle that for that commit
Basil
@nadar
Hi all. Are there any recommendations/examples on how to HOT RELOAD (LIVE RELOAD) a reactphp application? An example with docker-compose would be very nice. Thanks for the help
Cees-Jan Kiewiet
@WyriHaximus
hey @nadar
@nadar in a nutshell: reactive-apps/skeleton@2c63249
Basil
@nadar
(btw., nice that the community channel is finally on gitter :-D)
Anthony V
@fascinated
hello, i am trying to connect to a websocket server via a socks proxy, with ratchetphp/Pawl#101 for inspiration, but i think i am missing a step (or have my connector objects mixed up): https://pastebin.com/QY13J1TX

I get [below], which is making me think that, but haven't found another approach yet

PHP Fatal error: Uncaught Error: Function name must be a string in socket_proxy.php:45
Stack trace:

#0 socket_proxy.php(54): connect_socket(Object(Clue\React\Socks\Client), Object(React\EventLoop\StreamSelectLoop), Object(Closure))

#1 {main}

thrown in socket_proxy.php on line 45

Cees-Jan Kiewiet
@WyriHaximus
@fascinated not sure what to do with that error as your error messages don't seem to match up with lines in the file
Anthony V
@fascinated
er, sorry. i think something got moved. the fatal error is on the " $connector('wss://cooler.cloud/magic/')" line 46 (not 45).
Cees-Jan Kiewiet
@WyriHaximus
Which PHP version are you on?
Anthony V
@fascinated
7.2.24
Cees-Jan Kiewiet
@WyriHaximus
Doh
Try $connector->connect('wss://cooler.cloud/magic/') instead of $connector('wss://cooler.cloud/magic/'): https://github.com/clue/reactphp-socks/blob/master/src/Client.php#L115
Anthony V
@fascinated
oh hmm, the fatal is gone but now i am getting the "Could not connect: Invalid target URI specified" message from that ->connect method. Going to read that src + the examples a bit more, i didn't see that whole examples set when I found that mini-Pawl example above.
Anthony V
@fascinated
oh i see, it seems that the correct approach is similar to https://github.com/clue/reactphp-socks/blob/master/examples/02-https.php (but with the ratchet connector involved). i'll post a more complete example on that Pawl issue thread.
so this is the working version: https://pastebin.com/QY13J1TX. the ->connect vs ->() issue seems to have come up because of the differences in the ratchet connector vs. normal socket connector (if i am getting this correctly)
Basil
@nadar
image.png
@WyriHaximus When i run the php-watcher command in docker context i always get this message
Lochemem Bruno Michael
@ace411
Hello @nadar. Are you attempting to run the software on Windows?
Basil
@nadar
hey @ace411 no, i run inside a docker php:7.3-cli container
Lochemem Bruno Michael
@ace411
Okay. I don't have a lot of Docker knowledge - still learning how to use it, to be honest. @WyriHaximus is the best man to help with this sort of problem.
Anthony V
@fascinated
thanks for pointing me in the right direction @WyriHaximus! as an aside, is it best to run the various react/ratchet stuff on 7.3? I sort of assumed there isn't a big diff between 7.2-7.3.
Lochemem Bruno Michael
@ace411
@nadar Do you have ext-pcntl installed in your Docker PHP environment?
Basil
@nadar
Hey @ace411 indeed, i believe the default php 7.3 cli images don't have the extension installed. thanks for your help!
Christian Lück
@clue
@fascinated Happy to hear you're got this sorted out already! Yes, the ratchet/socket/socks "connector" lingo can be confusing, so there's definitely room for improvement :+1:
Anthony V
@fascinated
i think i mostly didn't understand this sort of Connector->Client->Connector object passing that's needed for the ratchet proxy stuff to work. but in retrospect it makes a lot of sense. hope my snippet helps others in the future!
Cees-Jan Kiewiet
@WyriHaximus
@nadar created these images for that: https://hub.docker.com/r/wyrihaximusnet/php
They come with ext-uv, ext-pcntl, and on the ZTS images with ext-parallel
Christian Lück
@clue
@fascinated Definitely going to help others, thanks for sharing your working SOCKS proxy code in ratchetphp/Pawl#101 :clap:
Christian Lück
@clue
@fascinated ReactPHP doesn't require a specific PHP version, but it usually works best on the latest PHP versions :+1:
Marc Morera
@mmoreram
do you have any experience with blackfire in reactPHP?
is there any magic trick on how to make that works?
Anthony V
@fascinated
has anyone written a style guide about some parts of react usage? for ex, i am using https://github.com/reactphp/stream to output to stdout from various async parts of a script. right now, i just do this by declaring the WritableResourceStream for STDOUT as a global, and access this global all over the place;.. is that a popular approach?
Cees-Jan Kiewiet
@WyriHaximus
@fascinated using a PSR-11 container myself to manage my dependencies: PHP-DI
That injects it in the classes using them so I don't have to global everything
Anthony V
@fascinated
oh nice, i'll check this out. yeah, that sounds better already
Cees-Jan Kiewiet
@WyriHaximus
You'd need a starting point, but you can build up the container first and then get the initial class from it to get started
Marc Morera
@mmoreram
Hey people! New LoopRun conference website is live - https://loop-run.io
As you can see, ReactPHP is an important part of the party... so...
Any feedback will be appreciated!
Chris Boden
@cboden
Awesome!
Christian Lück
@clue
@mmoreram That's so cool, really looking forward to that conference, to talk about @ReactPHP and also to discuss with other projects in the event-driven/async PHP community! :muscle:
Really love this idea, I see some Swoole representatives will also be there, do you know anything about Amp? /ping @kelunik