-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Upgrade to llhttp 9.2 #8146
Upgrade to llhttp 9.2 #8146
Conversation
@pajod Is test_parse_unusual_request_line and test_http_request_parser_utf8_request_line meant to be passing now? Still seems to be xfailing. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #8146 +/- ##
==========================================
- Coverage 97.51% 97.51% -0.01%
==========================================
Files 107 107
Lines 32743 32733 -10
Branches 3829 3824 -5
==========================================
- Hits 31930 31920 -10
Misses 612 612
Partials 201 201
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@Dreamsorcerer Expected and already the case when I introduced the other 2. Empty field name-intolerant llhttp just makes those 2 pass. |
Backport to 3.9: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 0ec65c0 on top of patchback/backports/3.9/0ec65c0f4dc08d027f659256b09ae9cff10ab404/pr-8146 Backporting merged PR #8146 into master
🤖 @patchback |
Backport to 3.10: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 0ec65c0 on top of patchback/backports/3.10/0ec65c0f4dc08d027f659256b09ae9cff10ab404/pr-8146 Backporting merged PR #8146 into master
🤖 @patchback |
(cherry picked from commit 0ec65c0)
(cherry picked from commit 0ec65c0)
No description provided.