Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

http_parser no longer maintained #132

Closed
seghcder opened this issue Dec 3, 2020 · 2 comments
Closed

http_parser no longer maintained #132

seghcder opened this issue Dec 3, 2020 · 2 comments

Comments

@seghcder
Copy link

seghcder commented Dec 3, 2020

Doing some dependency updates and saw this - nodejs/http-parser#522:

I'm moving on and as the last (semi-)active maintainer, that means http-parser is now effectively unmaintained.
It's a stable project for a stable protocol and I don't expect many issues but you should take the above into account when evaluating whether http-parser is the right choice for you.
For new projects and projects looking to migrate, I recommend https://github.com/nodejs/llhttp. Similar API, feature parity, well maintained and in active use.

Perhaps important given the security context here - #109 (comment)

Per the quote, it's probably not an immediate concern but something to potentially factor into 0.7x thinking?

@eao197
Copy link
Member

eao197 commented Dec 3, 2020

Hi! Thanks for reporting this!

I didn't know about http-parser maintenance problems. It's a pity.

However, it opens at least two possibilities: one is the switching to an alternative (like llhttp mentioned above, or picohttpparser, or something else), another is to develop our own header-only HTTP-parser (by using restinio::easy_parser that can simplify such task significantly). And if we want to move to support not only HTTP/1.1, but also HTTP/2 and HTTP/3, the own parser may be a good scenario to go.

Anyway, I have to spend the nearest week or two to close some SObjectizer-related tasks. Hope to return back to RESTinio after that.

It leaves some time for me to think about choices.

PS. The morning sun never lasts the day. :(

@eao197
Copy link
Member

eao197 commented Nov 13, 2023

llhttp is used since v.0.7.0

@eao197 eao197 closed this as completed Nov 13, 2023
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

No branches or pull requests

2 participants