Consider adding `pause` node

Question

Consider adding `pause` node

indutny opened this issue 7 years ago · 4 comments

There's http_parser_pause in http-parser repo. Not sure if we want to have it, or how it should work, but we should consider adding similar API.

Answer 1 · 2018-02-26T23:41:57.000Z

There is a error_pos field in state now. It could be used for resuming parsing.

Answer 2 · 2018-02-27T04:49:28.000Z

Looks like we got it working in c986905

Answer 3 · 2018-02-27T07:05:05.000Z

As it is right now, only the span callback can actually pause consistently. Invoke has no way to do it properly. Perhaps we need p.pause(code, reason) node.

Answer 4 · 2018-02-27T07:14:32.000Z

Maybe p.once(p.error(PAUSE_CODE, reason))?