Skip to content

Pause/resume on request not working #84

Closed

Description

@nick4fake

Hi,

Why do we have this code in Server class?

$request->on('pause', function () use ($conn) {
    $conn->emit('pause');
});
$request->on('resume', function () use ($conn) {
    $conn->emit('resume');
});

I have changed emit's to correct methods (pause/resume), like this:

$request->on('pause', function () use ($conn) {
    $conn->pause();
});
$request->on('resume', function () use ($conn) {
    $conn->resume();
});

And now pausing works.

Activity

  1. clue commented on Nov 22, 2016

    @clue
    Member

    Thanks for reporting! Do you feel like filing a PR and adding some tests to verify this? Thanks! 👍

  2. nick4fake commented on Nov 22, 2016

    @nick4fake
    Author

    @clue Ok, will do that tomorrow

  3. added a commit that references this issue on Nov 29, 2016
    f126189
  4. nick4fake commented on Nov 29, 2016

    @nick4fake
    Author

    @clue sorry for delaying the fix. Created pull review.

    Is there any roadmap for merging all those pull request? There are 25 PRs now.

  5. added this to the v0.4.4 milestone on Feb 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions