Simon Frings
Simon Frings
Hey @danepowell, I think this is a nice addition to the documentation 👍 This is also a very specific example, which is fine because it matches your use case, but...
Hey @ClosetMonkey, looks like a cool project, thanks for sharing! 👍 This ticket encourages me to start a discussion about adding "GitHub Discussions" to all ReactPHP repositories. There's currently only...
@51imyy I accidentally marked the first half of the examples (in `README.md` and docblocks) in my last review which now causes that half to be missing 😅
> So the real intention was only to change `var_dump($response->getHeaders());` to `var_dump($response);` right @51imyy Yep, I messed my suggestion up.
Hey @lcobucci, already a win if this helps you with your project 👍 Whether this will be a new feature or not depends on how many people need this for...
Hey @dinooo13 , thanks for looking into this one :+1: This is an interesting topic and the approach looks nice. It may not seem like it but I think this...
@dinooo13 Looking good, but I still think the logic for 303 should use the default behavior (HEAD stays HEAD and GET stays GET). The logic behind your code works great,...
Seems like this is the way to go then :+1: I also found this inside the [Guzzle documentation](https://guzzle3.readthedocs.io/http-client/http-redirects.html#redirects-and-non-repeatable-streams), throwing an exception in this case could be the answer for this...
@WyriHaximus Could be a sign, but I am here anyway ^^ @dinooo13 Changes are looking great, the last thing I want to mention is the amount of commits for this....
Ping @clue, this is ready for review 👍