security: fix URL truncation issue - #144
Merged
fantix merged 1 commit intoMay 25, 2026
Merged
Conversation
Copy link
Copy Markdown
Member
Sorry, something went wrong.
Copy link
Copy Markdown
Member
Author
|
@Kludex I feel like this would be an acceptable patch to stop the bleeding for now? (In the long run, we should migrate away from the no-longer-maintained http-parser and land one of the Rust approaches) |
Sorry, something went wrong.
Copy link
Copy Markdown
Contributor
|
Yes, this is enough. Thanks so much! 🙏 |
Sorry, something went wrong.
Merged
pull Bot
pushed a commit
to sysfce2/python-httptools
that referenced
this pull request
May 25, 2026
Changes ======= * Add http-parser and llhttp licenses into the wheels (MagicStack#135) (by @justeph in c398a15) * Mark cython module as free-threading compatible (MagicStack#139) (by @kumaraditya303 in 28d1db1) * Fix all typing issues (MagicStack#134) (by @Kludex in a9bda0e) * Bump llhttp to 9.4.1 (MagicStack#145) (by @fantix in e3e8d71) * Security: fix URL truncation issue (MagicStack#144) (by @fantix in a0283f0 for MagicStack#142) * Allow building with latest setuptools (MagicStack#138) (by @OldManYellsAtCloud in c403ad1)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #142