Skip to content

Conversation

@ShogunPanda
Copy link
Contributor

Fixes #176.

@ShogunPanda ShogunPanda requested a review from mcollina August 26, 2022 17:35
Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be documented?

@ShogunPanda
Copy link
Contributor Author

@mcollina I've added full documentation of the API. What about now?

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ShogunPanda ShogunPanda merged commit 7094b97 into nodejs:main Sep 7, 2022
@ShogunPanda ShogunPanda deleted the on-reset branch September 7, 2022 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

How to handle multiple requests in one packet with llhttp

2 participants