jshttp / on-finished Public

Notifications You must be signed in to change notification settings
Fork 46
Star 397

Code
Issues 5
Pull requests 7
Actions
Projects
Security
Insights

Additional navigation options

Code
Issues
Pull requests
Actions
Projects
Security
Insights

Pull requests: jshttp/on-finished

Labels 20 Milestones 0

New pull request New

7 Open 16 Closed

Author

Filter by author

Label

Filter by label

Use alt + click/return to exclude labels

or ⇧ + click/return for logical OR

Projects

Filter by project

Milestones

Filter by milestone

Reviews

Filter by reviews

No reviews Review required Approved review Changes requested

Assignee

Filter by who’s assigned

Assigned to nobody

Sort

Sort by

Newest Oldest Most commented Least commented Recently updated Least recently updated Best match

Most reactions

Pull requests list

remove async_hooks workaround for older node versions semver-major

#59 opened May 9, 2025 by Phillip9587

Loading…

refactor: use stream.finished() instead of custom implementation semver-major

#56 opened Apr 29, 2025 by Phillip9587

Loading…

use writableEnd instead finished

#54 opened Apr 13, 2025 by bjohansebas

Loading…

build: Node.js 19+

#53 opened Jan 15, 2025 by bjohansebas

Loading…

Refactor wrap function for better readability and efficiency

#51 opened Nov 8, 2024 by Ayoub-Mabrouk

Loading…

socket might never be emitted needs tests pr

#32 opened Jun 24, 2019 by ronag

Loading…

response is only finished if socket is detached needs tests pr

#31 opened Jun 24, 2019 by ronag

Loading…

ProTip! Type g i on any issue or pull request to go back to the issue listing page.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly