-
Notifications
You must be signed in to change notification settings - Fork 322
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
Incompatibility with JRuby v9.2.0.0 due to problem with http_parser.rb v0.6.0. #475
Comments
We have plans on switching http parser. If I will have some time this weekned - will try to work on that. |
Def interested in this working with jruby 9.2 asap, following. |
I have cut 5.0.0 pre release version of gem with switch to updated http parser. This should be fixed now. |
Oh nice! Thanks! Are there release notes or a URL to a relevant PR or commit about the changed http parser? I'm curious about the details, if what's used in MRI and Jruby both changed, to the same thing, what it is, etc.; since previous http parser (at least in jruby?) was a third-party thing that ended up unmaintained, interested in looking at social context of the new parser and it's maintainance story. |
This is the commit: f1db84f Right now we switched to maintained ffi-based http parser: https://github.com/cotag/http-parser |
Thanks! So nice to see http.rb getting maintenance! |
See http_parser.rb bug report.
The text was updated successfully, but these errors were encountered: