Httptools trims long urls
See original GitHub issuePrompted as part of the discussion in #344 , it seems httptools doenst handle well long url, see #925 for a failing test
I can say that for my case the issue is not URLs longer than 65,535 characters, but rather that on_url
is being called multiple times for whatever reason: https://github.com/nodejs/http-parser#parsing-urls.
_Originally posted by @malthejorgensen in https://github.com/encode/uvicorn/issues/344#issuecomment-752074479_
Issue Analytics
- State:
- Created 3 years ago
- Comments:5 (4 by maintainers)
Top Results From Across the Web
Developers - Httptools trims long urls - - Bountysource
Prompted as part of the discussion in #344 , it seems httptools doenst handle well long url, see #925 for a failing test....
Read more >URL confusion vulnerabilities in the wild: Exploring parser ...
In this post, we will take a look at the history of URLs, explore possible sources of URL parser confusion, run through an...
Read more >Trim URL Tool: Trim URLs to Their Root Domain - ClickMinded
This simple tool will let you trim a bunch of URLs to the root domain and standardize the prefixes. Makes your analysis easier....
Read more >URL Parsing-Library Bugs Allow DoS, RCE, Spoofing & More
“Since URI.parse trims off two slashes, the resulting URL will be ///evil.com. Whenever the server redirects the user to this URL, ///evil.com ...
Read more >INET Framework for OMNeT++/OMNEST: inet::httptools ...
inet::httptools::HttpBrowserBase Class Referenceabstract ... Requires the network topology to be set up – routers, links, etc. ... inet::httptools::trim.
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
jfyk, @victoraugustolls PR was merged, and we have a PR using the new httptools version: https://github.com/encode/uvicorn/pull/1024
Cool, I subscribed to it thanks for the heads up @victoraugustolls !